Skip to content

Conversation

@MrCasCode
Copy link

Some providers send a non-standard typing, so add an alias to allow decoding capital version

Some providers send a non-standard typing, so add an alias to allow decoding capital version
@tomusdrw
Copy link
Contributor

tomusdrw commented Nov 8, 2021

Hey @MrCasCode, thank you for the contribution. We are trying to implement the JSON-RPC specification strictly (see also #595, #448, #642) and the spec states:

All member names exchanged between the Client and the Server that are considered for matching of any kind should be considered to be case-sensitive. The terms function, method, and procedure can be assumed to be interchangeable.

Hence I'm going to close the PR without merging - adding support for arbitrary implementations that don't match the spec is not something we want to support long term - feel free to reach out to the providers you've mentioned and have them fix the member case.

Alternatively please consider implementing a more generic solution of non-strict mode, given in such mode we could support JSONRPC member I don't see a reason why JSONrpc or JsOnRpC should be rejected - i.e. we should ignore casing in members and also allow arbitrary extra members as described in #595.

I hope you'll understand the decision, but feel free to re-open or discuss further.

@tomusdrw tomusdrw closed this Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants