Skip to content

Conversation

eddeh
Copy link
Contributor

@eddeh eddeh commented Sep 2, 2025

Fixed the YAML configuration structure for defining in-memory users in the test environment. The previous configuration was missing the required providers key, which would result in a configuration error.

Fixed the YAML configuration structure for defining in-memory users in the test environment. The previous configuration was missing the required providers key, which would result in a configuration error.
@carsonbot carsonbot added this to the 7.4 milestone Sep 2, 2025
@carsonbot carsonbot changed the title Fix incorrect security configuration structure in testing documentation Fix incorrect security configuration structure in testing documentation Sep 2, 2025
@xabbuh
Copy link
Member

xabbuh commented Sep 4, 2025

Do we actually need this when #21338 is merged up?

@javiereguiluz
Copy link
Member

@xabbuh is right. @eddeh you only have to make the fix in the oldest still maintained Symfony branch that contains the bug ... and we later merge that fix up to all the other still maintained branches. We just did that, so we can close this one. Thanks!

@eddeh
Copy link
Contributor Author

eddeh commented Sep 4, 2025

@xabbuh @javiereguiluz @OskarStark

Thanks for the explanation and for merging the fix. This was my first PR, so I wasn’t familiar with the branching procedure for documentation fixes. I appreciate you taking care of merging it up to the other maintained branches.
I’ll make sure to target the oldest maintained branch (in this case 6.4) that contains the issue in future PRs. Thanks

@xabbuh
Copy link
Member

xabbuh commented Sep 4, 2025

No need to worry. :) We can usually easily switch branches while merging and closing a PR also isn't hard work. It simply spares you some time if you don't have to prepare several PRs. 😄

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

Successfully merging this pull request may close these issues.

5 participants