-
Notifications
You must be signed in to change notification settings - Fork 433
Removed multiple public apis #2888
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.
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
Conversation
revised how exceptions are created
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pulled locally and ran the tests, the .txt
look good
would be good to get another set of eyes though
Can you link when these APIs were introduced? Or otherwise show some diff showing that these specific areas were touched? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @brentschmaltz
LGTM. Internal SecurityTokenException.ValidationError was removed by #2818, but looks like it wasn't used anywhere. |
We inadvertently shipped some public api's that are not ready, some were breaking modifications.
We removed the interface ISecurityTokenException (breaking)
SecurityTokenException.SetValidationError(ValidationError validationError) was changed internal
public class ValidationError was changed to internal
classes AudienceValidationError and LifetimeValidationError were modified to accept specific values in ctor removing the need for AdditionalInfo.