Skip to content

Conversation

scottfrederick
Copy link
Contributor

Prior to this commit, active profiles were being added to the Spring Boot application environment by setting the spring.profiles.active property. This could result in profiles getting parsed differently than other uses of @ActiveProfiles. Setting the profiles directly in the Environment prevents this parsing.

Fixes #19537

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 7, 2020
@wilkinsona wilkinsona changed the title Add profiles directly to the application Environment Profile with comma in name does not work in tests Jan 7, 2020
@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 7, 2020
@wilkinsona wilkinsona added this to the 2.1.x milestone Jan 7, 2020
Copy link
Member

@wilkinsona wilkinsona left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Prior to this commit, active profiles were being added to the Spring Boot application environment by setting the `spring.profiles.active` property. This could result in profiles getting parsed differently than other uses of `@ActiveProfiles`. Setting the profiles directly in the `Environment` prevents this parsing.

Fixes spring-projects#19537
mbhave pushed a commit that referenced this pull request Jan 7, 2020
Prior to this commit, active profiles were being added to the Spring Boot
application environment by setting the `spring.profiles.active` property.
This could result in profiles getting parsed differently than other uses of `@ActiveProfiles`.
Setting the profiles directly in the `Environment` prevents this parsing.

See gh-19556
@mbhave mbhave closed this in 6d6bbfb Jan 7, 2020
@mbhave mbhave modified the milestones: 2.1.x, 2.1.12 Jan 7, 2020
@scottfrederick scottfrederick deleted the gh-18482 branch January 7, 2020 21:44
@wilkinsona
Copy link
Member

This change has been reverted in 2.1.13 as it introduced a regression. See #19788 for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants