Skip to content

Comments

ThrowIfNullOrWhiteSpace for type in IdentityProvider#1675

Merged
josephdecock merged 1 commit intoDuendeSoftware:mainfrom
SimonCropp:throw-ThrowIfNullOrWhiteSpace-for-type-in-IdentityProvider
Jan 28, 2025
Merged

ThrowIfNullOrWhiteSpace for type in IdentityProvider#1675
josephdecock merged 1 commit intoDuendeSoftware:mainfrom
SimonCropp:throw-ThrowIfNullOrWhiteSpace-for-type-in-IdentityProvider

Conversation

@SimonCropp
Copy link
Contributor

I assume empty and whitespace is invalid for type?

Copy link
Member

@josephdecock josephdecock left a comment

Choose a reason for hiding this comment

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

This looks fine to me - I can't picture anyone ever correctly configuring an IdentityProvider with a type that was just white space.

@josephdecock josephdecock merged commit f753fe1 into DuendeSoftware:main Jan 28, 2025
@josephdecock josephdecock added this to the is-7.2.0 milestone Jan 28, 2025
@SimonCropp SimonCropp deleted the throw-ThrowIfNullOrWhiteSpace-for-type-in-IdentityProvider branch January 28, 2025 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/products/is IdentityServer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants