Skip to content

AuthorizePayloadsSpec uses AUTHENTICATION for AuthorizationPayloadInterceptor #7434

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
Closed
rwinch opened this issue Sep 13, 2019 · 2 comments · Fixed by #7450
Closed

AuthorizePayloadsSpec uses AUTHENTICATION for AuthorizationPayloadInterceptor #7434

rwinch opened this issue Sep 13, 2019 · 2 comments · Fixed by #7450
Assignees
@rwinch
Labels
in: config An issue in spring-security-config type: bug A general bug
Milestone
5.2.0

Comments

@rwinch
Copy link
Member

rwinch commented Sep 13, 2019 •
edited
Loading 

result.setOrder(PayloadInterceptorOrder.AUTHENTICATION.getOrder());

should be

result.setOrder(PayloadInterceptorOrder.AUTHORIZATION.getOrder());
@rwinch rwinch added in: config An issue in spring-security-config type: bug A general bug status: ideal-for-contribution An issue that we actively are looking for someone to help us with labels Sep 13, 2019
@rwinch rwinch added this to the 5.2.0 milestone Sep 13, 2019
@eddumelendez
Copy link
Contributor

Hi @rwinch I can take this issue. Some friends are starting contributing so I can work this one with them. /cc @jascama

@jascama
Copy link
Contributor

jascama commented Sep 14, 2019

Sure, I can work on it

jascama added a commit to jascama/spring-security that referenced this issue Sep 17, 2019
@jascama
Fix AuthorizationPayloadInterceptor order using PayloadInterceptorOrd…
f049c6e 
…er.AUTHORIZATION

Fixes spring-projectsgh-7434
@jascama jascama mentioned this issue Sep 17, 2019
Merged
rwinch pushed a commit that referenced this issue Sep 24, 2019
@jascama @rwinch
Fix AuthorizationPayloadInterceptor order using PayloadInterceptorOrd…
ceab56f 
…er.AUTHORIZATION

Fixes gh-7434
@rwinch rwinch self-assigned this Sep 24, 2019
@rwinch rwinch removed the status: ideal-for-contribution An issue that we actively are looking for someone to help us with label Sep 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rwinch rwinch

Labels
in: config An issue in spring-security-config type: bug A general bug
Projects
None yet
Milestone
5.2.0
Development

Successfully merging a pull request may close this issue.

Fix AuthorizationPayloadInterceptor order using PayloadInterceptorOrd… jascama/spring-security
3 participants
@rwinch @eddumelendez @jascama