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

fix: proxy-test property sources to lambda handler ctx #1337

Merged

sdelamo merged 1 commit into 3.2.x from fix-proxy-test-context-property-sources

Apr 12, 2022

Contributor

sdelamo commented Apr 4, 2022 •

edited

Loading

Without this PR, you cannot use a @Requires annotation for an @Property - the typical spec.name pattern to avoid bean pollution. Because the api-proxy-test appliation context property sources are differnet than the MicronautLambdaHandler property sources.


          fix: proxy-test property sources to lambda handler ctx

5b45e62

sdelamo added the type: bug label

sonarqubecloud bot commented Apr 4, 2022

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication

sdelamo requested a review from graemerocher

April 4, 2022 11:29

Contributor Author

sdelamo commented Apr 11, 2022

@graemerocher can you review this?

graemerocher approved these changes

View reviewed changes

sdelamo merged commit 6a4bf9f into 3.2.x

sdelamo deleted the fix-proxy-test-context-property-sources branch

April 12, 2022 14:09

sdelamo added a commit that referenced this pull request


          fix: proxy-test property sources to lambda handler ctx (#1337)

edd5cd7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels