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

default headers will not override those set by user anymore #6

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