発端 GraphQL API の利用者から API クライアントのコードレビューに付き合ってほしいと言われたので出席したんですが、とんでもない実装になっていて腰を抜かしてしまったので、注意喚起しようかなと。 はじめに結論 クエリの一部にユーザ入力を文字列連結するのはやめましょう! 動的パラメータは variablesで指定しましょう。 variablesの指定例は以下を参照。 https://graphql.org/learn/serving-over-http/#post-request ポピュラーな GraphQL クライアントであれば、 variablesに当たる変数を渡すことができるインタフェースが用意されていると思います。 インジェクションが起きる仕組み 実装言語は何でも良いです。 以下のような API クライアント実装があるとします。 const userInput =
![GraphQL クライアントではクエリインジェクションに注意しよう - Qiita](https://cdn-ak-scissors.b.st-hatena.com/image/square/3acba4fa18a1205de828a91826b7309a6adb61b2/height=288;version=1;width=512/https%3A%2F%2Fqiita-user-contents.imgix.net%2Fhttps%253A%252F%252Fcdn.qiita.com%252Fassets%252Fpublic%252Farticle-ogp-background-412672c5f0600ab9a64263b751f1bc81.png%3Fixlib%3Drb-4.0.0%26w%3D1200%26mark64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZ3PTk3MiZoPTM3OCZ0eHQ9R3JhcGhRTCUyMCVFMyU4MiVBRiVFMyU4MyVBOSVFMyU4MiVBNCVFMyU4MiVBMiVFMyU4MyVCMyVFMyU4MyU4OCVFMyU4MSVBNyVFMyU4MSVBRiVFMyU4MiVBRiVFMyU4MiVBOCVFMyU4MyVBQSVFMyU4MiVBNCVFMyU4MyVCMyVFMyU4MiVCOCVFMyU4MiVBNyVFMyU4MiVBRiVFMyU4MiVCNyVFMyU4MyVBNyVFMyU4MyVCMyVFMyU4MSVBQiVFNiVCMyVBOCVFNiU4NCU4RiVFMyU4MSU5NyVFMyU4MiU4OCVFMyU4MSU4NiZ0eHQtYWxpZ249bGVmdCUyQ3RvcCZ0eHQtY29sb3I9JTIzMjEyMTIxJnR4dC1mb250PUhpcmFnaW5vJTIwU2FucyUyMFc2JnR4dC1zaXplPTU2JnM9MmM3NzY5ZDU5YzEwNzEzYmE2ZWQ2MzBiZDYzZGQzY2Q%26mark-x%3D142%26mark-y%3D57%26blend64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZoPTc2Jnc9NzcwJnR4dD0lNDB0YWtleW9kYSZ0eHQtY29sb3I9JTIzMjEyMTIxJnR4dC1mb250PUhpcmFnaW5vJTIwU2FucyUyMFc2JnR4dC1zaXplPTM2JnR4dC1hbGlnbj1sZWZ0JTJDdG9wJnM9OTU1MmNiY2U5NmUzZTNkZGFhNTQ5Mzg3NGI1OWFhZmE%26blend-x%3D142%26blend-y%3D486%26blend-mode%3Dnormal%26s%3D2e0278af84d258a393844623f28323ab)