request POST JSON data is not handled as expected

[petschki]

What I did:

I sent an XHR call to zope like this:

const resp = await fetch("@@backend", {
    headers: {
        "Content-Type": "application/json",
    },
    method: "POST",
    body: JSON.stringify({
        id: "some_id",
        _authenticator: "some_authenticator",
    }),
});

What I expect to happen:

If I want to read the request in @@backend browser view I expect:

self.request.form = {
    "id": "some_id",
    "_authenticator": "some_authenticator",
}

What actually happened:

You have to extract the body data manually via:

form_data = json.loads(self.request["BODY"])

What version of Python and Zope/Addons I am using:

python: 3.9
Zope: master
Plone: 6.0-dev

[d-maurer]

Peter Mathis wrote at 2022-6-2 02:52 -0700:

I sent an XHR call to zope like this: ``` const resp = await fetch("@@backend", { headers: { "Content-Type": "application/json", }, method: "POST", body: JSON.stringify({ id: "some_id", _authenticator: "some_authenticator", }), }); ... ### What I expect to happen: If I want to read the request in `@@backend` browser view I expect: ``` self.request.form = { "id": "some_id", "_authenticator": "some_authenticator", } ``` ### What actually happened: You have to extract the body data manually via: ``` form_data = json.loads(self.request["BODY"]) ```

Zope currently looks at `content-type` only in very rare cases: only to check for `XML-RPC` requests. I would not be happy to support `application/json` in the way you suggest: it works only when the body represents a structure (then there are names to bind to); it does not work for e.g. a string or an array (no names to bind to). I suggest, you use instead the `:json` type converter. With this, your example could look like: `await fetch("@@backend?params:json=" + JSON.stringify({...})` and you would get the JSON decoded value in `request["params"]`. A similar approach would work for POST requests. I believe that I already have seen a PR supporting the `:json` type converter: it might already be available. Should I be wrong, "zopefoundation/Zope#648" contains (among other things) such support. It has a good chance to arrive in Zope 6.

[mgedmin]

-1 for reusing request.form for this.

+1 for introducing a new request.json.