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

Invalid request type: Servlet3SecurityContextHolderAwareRequestWrapper. Only AwsProxyHttpServletRequest is supported #275

Closed
1103036128 opened this issue Aug 10, 2019 · 6 comments

Comments

@1103036128
Copy link

Hello, I have another problem, spring sercurity has made a mistake. I need your help.

  • Framework version: 1.3.2
  • Implementations: Spring Boot+Springsecurity

An error occurred when the loadUserByUsername to return user in the Security UserDetails Service
image
The CloudWatch error log are --
image
image
image
Please help me, thank you.

@1103036128
Copy link
Author

Here is my sercurity config --
image

@1103036128
Copy link
Author

image
How to convert security requests into AwsProxyHttpServletRequest requests

@1103036128
Copy link
Author

Now I've removed the Spring Security redirection.

@sapessi sapessi added this to the Release 1.4 milestone Aug 11, 2019
@sapessi
Copy link
Collaborator

sapessi commented Aug 11, 2019

Thanks for reporting this @1103036128. I'll look into it for the next release.

sapessi added a commit that referenced this issue Aug 14, 2019
@sapessi
Copy link
Collaborator

sapessi commented Aug 14, 2019

Hi @1103036128 - I have implemented a fix for this in the core branch. Let me know if you can test and whether it works for you - that was a deep rabbit hole! Thanks again for the report

sapessi added a commit that referenced this issue Aug 14, 2019
… on their registration's DispatcherTypes. Changed the default registration to include REQUEST and ASYNC types. This completes the changes for #275
@sapessi
Copy link
Collaborator

sapessi commented Sep 30, 2019

Release 1.4 is out on maven central. The release notes include a list of all the changes. Closing this issue.

@sapessi sapessi closed this as completed Sep 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants