Skip to content

Add annotation required to switch on constructor binding #18469

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
philwebb opened this issue Oct 1, 2019 · 1 comment
Closed

Add annotation required to switch on constructor binding #18469

philwebb opened this issue Oct 1, 2019 · 1 comment
Assignees
@philwebb
Labels
type: enhancement A general enhancement
Milestone
2.2.0.RC1

Comments

@philwebb
Copy link
Member

philwebb commented Oct 1, 2019

The heuristics to work out when a @ConfigurationProperties class is bound as a bean vs when constructor binding is used are currently not clear. Whilst it might be possible to deduce them in the future, requiring a specific signal that constructor binding is needed should be added for 2.2.
@philwebb philwebb added the type: enhancement A general enhancement label Oct 1, 2019
@philwebb philwebb added this to the 2.2.x milestone Oct 1, 2019
@philwebb philwebb mentioned this issue Oct 2, 2019
Closed
@philwebb philwebb changed the title Add annotation required to switch on contructor binding Add annotation required to switch on constructor binding Oct 2, 2019
@philwebb
Copy link
Member Author

philwebb commented Oct 2, 2019

See also #17831, #18322 and #17035

@philwebb philwebb self-assigned this Oct 2, 2019
@philwebb philwebb modified the milestones: 2.2.x, 2.2.0.RC1 Oct 2, 2019
This was referenced Oct 2, 2019
Closed
Closed
@philwebb philwebb mentioned this issue Oct 2, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philwebb philwebb

Labels
type: enhancement A general enhancement
Projects
None yet
Milestone
2.2.0.RC1
Development

No branches or pull requests
1 participant
@philwebb