Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.2] Allow HTTP-RPC with empty response #630

Merged
merged 1 commit into from
Jul 11, 2022
Merged

[3.2] Allow HTTP-RPC with empty response #630

merged 1 commit into from
Jul 11, 2022

Conversation

heifner
Copy link
Member

@heifner heifner commented Jul 9, 2022

Current http request handlers always return the json string converted from fc::variant, it is not allowed to make HTTP-RPC with empty response. (At least, it will have "null" in the case of fc::variant())

I [conr2d] am writing jsonrpc_plugin working upon http_plugin, and when request is given without "id" (it's called notification), JSONRPC server must not reply to it. This patch makes it possible to create API with empty response.

Backport of EOSIO/eos#9633

Resolves #435

Thanks @conr2d
Co-Authored-By: conr2d@gmail.com

@heifner heifner added the OCI OCI working this issue... label Jul 9, 2022
@oschwaldp-oci oschwaldp-oci self-requested a review July 11, 2022 17:27
@heifner heifner merged commit 2769a23 into main Jul 11, 2022
@heifner heifner deleted the GH-435-http-empty branch July 11, 2022 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCI OCI working this issue...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Backport: Allow HTTP-RPC with empty response
2 participants