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

Property Resolving in @RequestMapping for Spring Boot #1375

Closed
wants to merge 2 commits into from

Conversation

lyca
Copy link
Contributor

@lyca lyca commented Oct 4, 2020

When I provided the solution for #361 I had a special use case that got solved. I thought, the solution would help also in Spring Boot, but I was wrong -> https://stackoverflow.com/questions/22167912/contextloader-getcurrentwebapplicationcontext-always-returns-null

So I worked on another solution for the problem and open this PR to discuss this one.

  • should work in all Spring Boot web applications
  • EnvironmentContextFilter does not need its own configuration class, I did not know, where to put it

- should work in all Spring Boot web applications
- the EnvironmentConfiguration is a bit too much
- EnvironmentContextFilter should be added to some of the other configuration classes
@lyca lyca changed the title Feature/filter thread local Property Resolving in @RequestMapping for Spring Boot Oct 31, 2020
@adriendengreville
Copy link

Any chance of merging this PR for a future release, as it would fix a regression when using Spring Boot ? (please see referenced issue)

@lyca
Copy link
Contributor Author

lyca commented Dec 10, 2020

Created an easier solution in #1411

@lyca lyca closed this Dec 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants