8000 default headers will not override those set by user anymore by digitronac · Pull Request #6 · zendframework/zend-json-server · GitHub
[go: up one dir, main page]

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

0