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

Stateless Authorization Handler Issue in 1.6.26 #556

Open
soaapp opened this issue Apr 7, 2021 · 4 comments
Open

Stateless Authorization Handler Issue in 1.6.26 #556

soaapp opened this issue Apr 7, 2021 · 4 comments
Labels
1.6.x Merge to 1.6.x Branch help wanted question

Comments

@soaapp
Copy link

soaapp commented Apr 7, 2021

Since updating to 1.6.26 for an application, the functionality of handling authorization code redirect changed. (previously used version 1.6.10) Please provide description on new behaviour and guidance on migrating to this new functionality.

@soaapp soaapp added help wanted question 1.6.x Merge to 1.6.x Branch labels Apr 7, 2021
@BalloonWen
Copy link
Contributor

could you please provide more info like the error loggin?

@stevehu
Copy link
Contributor

stevehu commented Apr 8, 2021

@soaapp I don't remember we made any change in the 1.6.x as it is a maintained branch. If there are any changes, it would be a security fix. Would you be able to compare the two tags and let us know what has been changed?

@adbatesonwork
Copy link

I think it was narrowed down to this commit: networknt/light-spa-4j@49d3ece where it changed the response behaviour from going to the redirect uri to returning a JSON payload including the redirect uri among other things.

@stevehu
Copy link
Contributor

stevehu commented Apr 9, 2021

@adbatesonwork The change should only be limited to the 2.x branch. If 1.6.x is changed, we should roll it back. Could you please open a PR to get it updated? Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.6.x Merge to 1.6.x Branch help wanted question
Projects
None yet
Development

No branches or pull requests

4 participants