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

Avoid default Accept header from HttpUrlConnection in RestTemplate with put and delete #23740 #23855

Closed
wants to merge 2 commits into from

Conversation

@sap-nocops
Copy link

sap-nocops commented Oct 23, 2019

Fixes #23740

@pivotal-issuemaster

This comment has been minimized.

Copy link

pivotal-issuemaster commented Oct 23, 2019

@sap-nocops Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster

This comment has been minimized.

Copy link

pivotal-issuemaster commented Oct 23, 2019

@sap-nocops Thank you for signing the Contributor License Agreement!

Lorenzo Torracchi
@sap-nocops sap-nocops changed the title Avoid default Accept header from HttpUrlConnection in RestTemplate wi… Avoid default Accept header from HttpUrlConnection in RestTemplate with put and delete #23740 Oct 23, 2019
@rstoyanchev rstoyanchev self-assigned this Nov 7, 2019
@rstoyanchev rstoyanchev added this to the 5.2.2 milestone Nov 7, 2019
rstoyanchev added a commit that referenced this pull request Nov 8, 2019
Apply default Accept header for HttpUrlConnection only.

See gh-23855
rstoyanchev added a commit that referenced this pull request Nov 8, 2019
@rstoyanchev

This comment has been minimized.

Copy link
Contributor

rstoyanchev commented Nov 8, 2019

@sap-nocops thanks for the pull request! I've modified it slightly to apply the Accept header default to requests through HttpUrlConnection only.

@rstoyanchev rstoyanchev closed this Nov 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants
You can’t perform that action at this time.