Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Return removed updateRequest method from OidcConfig #5840

Closed
Verdent opened this issue Jan 12, 2023 · 1 comment
Closed

Return removed updateRequest method from OidcConfig #5840

Verdent opened this issue Jan 12, 2023 · 1 comment
Assignees
@Verdent
Labels
2.x Issues for 2.x version branch bug Something isn't working P1 security
Milestone
2.5.6

Comments

@Verdent
Copy link
Member

Verdent commented Jan 12, 2023

Method updateRequest was removed by mistake when doing OIDC multi-tenant implementation. Removal of this method creates backwards incompatible change. This method should be returned as deprecated to follow standard deprecation process.
@Verdent Verdent added bug Something isn't working security labels Jan 12, 2023
@Verdent Verdent self-assigned this Jan 12, 2023
@Verdent Verdent added the P1 label Jan 12, 2023
@github-actions github-actions bot added the 2.x Issues for 2.x version branch label Jan 12, 2023
This was referenced Jan 12, 2023
Closed
Open
@Verdent Verdent mentioned this issue Jan 12, 2023
Merged
@barchetta barchetta added this to the 2.5.6 milestone Jan 12, 2023
@Verdent
Copy link
Member Author

Verdent commented Jan 13, 2023

fixed

@Verdent Verdent closed this as completed Jan 13, 2023
@m0mus m0mus added this to Backlog Aug 12, 2024
@m0mus m0mus moved this to Closed in Backlog Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Verdent Verdent

Labels
2.x Issues for 2.x version branch bug Something isn't working P1 security
Projects
Backlog
Archived in project
Milestone
2.5.6
Development

No branches or pull requests
2 participants
@Verdent @barchetta