Skip to content

Comments

fix: Correctly binds AuthUsersConfig to AnonymousIdp#4758

Merged
marcelomendoncasoares merged 1 commit intoserverpod:mainfrom
craiglabenz:anonymous-idp-hotfix
Feb 23, 2026
Merged

fix: Correctly binds AuthUsersConfig to AnonymousIdp#4758
marcelomendoncasoares merged 1 commit intoserverpod:mainfrom
craiglabenz:anonymous-idp-hotfix

Conversation

@craiglabenz
Copy link
Contributor

Passes a dropped parameter through to AnonymousIdp.

Without this, the AnonymousIdp does not call any configured before/after AuthUserCreated hooks.

Pre-launch Checklist

  • I read the Contribute page and followed the process outlined there for submitting PRs.
  • This update contains only one single feature or bug fix and nothing else. (If you are submitting multiple fixes, please make multiple PRs.)
  • I read and followed the Dart Style Guide and formatted the code with dart format.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///), and made sure that the documentation follows the same style as other Serverpod documentation. I checked spelling and grammar.
  • I added new tests to check the change I am making.
  • All existing and new tests are passing.
  • Any breaking changes are documented below.

Breaking changes

None.

@marcelomendoncasoares marcelomendoncasoares changed the title [fix] Correctly binds AuthUsersConfig to AnonymousIdp fix: Correctly binds AuthUsersConfig to AnonymousIdp Feb 23, 2026
Copy link
Collaborator

@marcelomendoncasoares marcelomendoncasoares left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for this fix, @craiglabenz! 🎉

@marcelomendoncasoares marcelomendoncasoares merged commit d004113 into serverpod:main Feb 23, 2026
73 of 74 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants