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.

Sign up for GitHub

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

Jump to bottom

Don't send body with 403 #10545

Merged
merged 1 commit into from
Jul 8, 2020
Merged

Don't send body with 403 #10545

merged 1 commit into from
Jul 8, 2020

Conversation

stuartwdouglas
Copy link
Member

In most cases the client is not a browser.

Fixes #5998

@stuartwdouglas
Don't send body with 403
59265e1 
In most cases the client is not a browser.

Fixes quarkusio#5998
@geoand geoand requested a review from sberyozkin July 8, 2020 04:47
@sberyozkin
Copy link
Member

I guess one can inject HttpHeaders into the mapper and check Accept but the browser will itself indicate that the resource is forbidden, so returning 403 only looks fine :-)

@sberyozkin sberyozkin added this to the 1.7.0 - master milestone Jul 8, 2020
@sberyozkin sberyozkin merged commit 0307360 into quarkusio:master Jul 8, 2020
@gsmet gsmet modified the milestones: 1.7.0 - master, 1.6.1.Final Jul 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sberyozkin sberyozkin sberyozkin approved these changes

Assignees
No one assigned
Labels
area/resteasy-classic area/smallrye
Projects
None yet
Milestone
1.6.1.Final
Development

Successfully merging this pull request may close these issues.

JaxRS Security wrong content-type
3 participants
@stuartwdouglas @sberyozkin @gsmet