Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Business Central MCP Server: string tool arguments are not JSON-escaped

オープン
#8,350 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
48/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
json

調査の方向性

Start by locating the Business Central MCP Server's Create_, Modify_ and bound-action request-body construction paths; the issue does not name source files or tests. Trace how decoded tool arguments become OData JSON, and check whether string values are serialized or inserted into a template and whether apostrophe escaping is applied outside URL expressions. Done means all write paths preserve the supplied strings and the injection example cannot set an extra property; add coverage for the listed cases.

索引モデルが issue の本文から書いたものです。

説明

accepted

Summary

The Business Central MCP Server does not pass string tool arguments to the API page unchanged. The
client sends a valid JSON-RPC request, but the value that reaches the record is different, or the
call fails. Reproduced with the standard customers API (API v2.0, page 30009) and the
Create_Customer_PAG30009 tool:

displayName sent Result
O'Brien Ltd customer created as O''Brien Ltd ΓÇö the apostrophe is doubled
"Acme" Corp call fails: Invalid JSON. A comma character ',' was expected in scope 'Object'.
Acme\Corp call fails: Invalid JSON. An unrecognized escape sequence '\C' was found in a JSON string value.
Acme\nCorp (backslash + n) customer created with a line break in the name
Caf\u00e9 Corp (backslash + u00e9) customer created as Café Corp

The same request through the plain OData API v2.0, without the MCP Server, creates O'Brien Ltd
correctly. So the data is changed in the MCP Server, not in the API page.

Because the values are not encoded, a " in one argument can also add properties to the request:
displayName = Inj", "phoneNumber": "123 creates a customer named Inj with phone number 123
(see Security impact).

O'Brien, L'Oréal and D'Angelo are ordinary customer names, and the apostrophe is common in
French, Italian, Ukrainian and other languages. Quotes and backslashes appear in names, addresses
and descriptions too. An MCP client cannot write such values. The apostrophe, \n and \u cases
fail silently: the tool reports success and the record holds a different value.

Environment

  • Business Central on-premises. initialize reports
    {'name': 'Microsoft Dynamics 365 Business Central', 'version': '29.0.55365.0', 'description': 'Microsoft Dynamics 365 Business Central MCP Server'}.
    The behaviour was the same on the version we ran before upgrading to 29.
  • MCP Server configuration CUSTOMERS: API Tools activated, Unblock Edit Tools on,
    APIV2 - Customers (page 30009, API v2.0) added with Allow Read, Create, Modify and Delete.
  • Transport: Streamable HTTP, Basic authentication, headers Company and ConfigurationName.
  • Client: a plain JSON-RPC client (Python json.dumps + urllib), so no MCP host is involved. The same results occur from Claude Code and VS Code.
  • The same happens with custom API pages, for both Modify_* (PATCH) and bound actions with a
    Text parameter.

Steps to reproduce

  1. initialize, then notifications/initialized.

  2. tools/call of Create_Customer_PAG30009. This is the exact request body produced by
    json.dumps, so it is valid JSON and the server receives displayName = O'Brien Ltd:

    {"jsonrpc": "2.0", "id": 2, "method": "tools/call", "params": {"name": "Create_Customer_PAG30009",
     "arguments": {"number": "MCPTEST1", "displayName": "O'Brien Ltd", "select": "number,displayName"}}}
    
  3. Read the customer back with List_Customers_PAG30009 (filter: number eq 'MCPTEST1').

Actual results

Every value was sent JSON-encoded by json.dumps, so the server received exactly the text in the
"sent" column. The stored value was read back with List_Customers_PAG30009.

Customer Path displayName sent Outcome displayName stored
MCPTEST1 MCP Create_Customer_PAG30009 O'Brien Ltd success O''Brien Ltd
MCPTEST2 MCP Create_Customer_PAG30009 "Acme" Corp Invalid JSON. A comma character ',' was expected in scope 'Object'. Every two elements in an array and properties of an object must be separated by commas. not created
MCPTEST3 MCP Create_Customer_PAG30009 Acme\Corp Invalid JSON. An unrecognized escape sequence '\C' was found in a JSON string value. not created
MCPTEST4 MCP Create_Customer_PAG30009 Acme\nCorp (backslash + n) success Acme + line break + Corp
MCPTEST5 MCP Create_Customer_PAG30009 Caf\u00e9 Corp success Café Corp
MCPTEST6 OData API v2.0 POST …/companies({id})/customers, no MCP O'Brien Ltd success O'Brien Ltd
MCPTEST7 MCP Create_Customer_PAG30009 \"Acme\" Corp (pre-escaped by the client) success "Acme" Corp

Raw tool result for MCPTEST2:

{"result": {"content": [{"type": "text", "text": "{\r\n  \"error\": {\r\n    \"code\": \"BadRequest\",\r\n    \"message\": \"Invalid JSON. A comma character \\u0027,\\u0027 was expected in scope \\u0027Object\\u0027. Every two elements in an array and properties of an object must be separated by commas.\"\r\n  }\r\n}"}], "isError": true}}

Security impact: JSON injection through a tool argument

Because the value is inserted into the request body without encoding, a " in one argument can
close the string and add further properties to the OData request. With the same tool:

Customer Path displayName sent Outcome Stored
MCPTEST8 MCP Create_Customer_PAG30009 Inj", "phoneNumber": "123 success displayName = Inj, phoneNumber = 123

A single string argument set a second field that the caller did not pass as an argument. In an
agent scenario the argument value often comes from untrusted content (a document, an e-mail, a web
page the agent processed), so this is a prompt-injection-to-data-injection path: text controls which
fields of the record are written. This is CWE-116 (Improper Encoding or Escaping of Output) leading
to CWE-74 (Injection).

Expected behaviour

Tool arguments are data and must reach the API page unchanged:

  1. MCP / JSON-RPC. tools/call carries arguments as a JSON object
    (MCP specification, Tools,
    JSON-RPC 2.0). After parsing the request, the server
    holds the decoded string values, for example O'Brien Ltd and "Acme" Corp. It must not
    interpret them again.
  2. Building the OData request body. The body must be produced by a JSON serializer, which escapes
    ", \ and control characters as required by
    RFC 8259, section 7, and not by inserting the
    values into a JSON template. A string property in an OData request body is a plain JSON string
    (OData JSON Format v4.01).
  3. No OData literal escaping in the body. Doubling ' is the escaping rule for string literals
    inside the URL, i.e. $filter expressions and key predicates
    (OData URL Conventions v4.01).
    It must be applied only there, never to values sent in a request body.
  4. Every write path. The same applies to Create_* (POST), Modify_* (PATCH) and bound actions
    (Text parameters in the action's request body).

Concretely, the customers above must be created as O'Brien Ltd, "Acme" Corp, Acme\Corp,
Acme\nCorp (backslash and n) and Caf\u00e9 Corp (the six characters as sent), exactly as the
OData API does without the MCP Server (MCPTEST6), and MCPTEST8 must be created with
displayName = Inj", "phoneNumber": "123 and no phone number.

Workaround

  • " and \: the client can JSON-escape the value itself (\", \\) before sending it
    (MCPTEST7). No MCP client does this on its own, and it will corrupt data once the server is fixed.
  • ': none. Through the MCP Server the apostrophe is always stored doubled.

Internal work item: AB#653294

主要言語
PowerShell
スター
886
フォーク
285
平均マージ
4時間 7分
マージ済み PR(30日)
1

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

microsoft/AL のほかの issue

microsoft/AL の issue をすべて見る

似ている issue

Backend & API Design の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。