Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 30, 2020. It is now read-only.

Conversation

digitronac
Copy link
Contributor

was unable to change default headers on json rpc client so i have submitted this PR.

also changed default headers for content type and accept to more appropriate "application/json-rpc" as per specification http://www.jsonrpc.org/historical/json-rpc-over-http.html#http-header

Content-Type SHOULD be 'application/json-rpc' but MAY be 'application/json' or 'application/jsonrequest'

The Accept MUST be specified and SHOULD read 'application/json-rpc' but MAY be 'application/json' or 'application/jsonrequest'.

@weierophinney weierophinney merged commit 27f359c into zendframework:master Apr 25, 2018
weierophinney added a commit that referenced this pull request Apr 25, 2018
…fied

default headers will not override those set by user anymore
weierophinney added a commit that referenced this pull request Apr 25, 2018
weierophinney added a commit that referenced this pull request Apr 25, 2018
weierophinney added a commit that referenced this pull request Apr 25, 2018
@weierophinney
Copy link
Member

Thanks, @digitronac!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants