Skip to content

Removing OAuthServiceExceptionMapper which isn't used anywhere#3128

Open
coheigea wants to merge 1 commit into
mainfrom
coheigea/remove-OAuthServiceExceptionMapper
Open

Removing OAuthServiceExceptionMapper which isn't used anywhere#3128
coheigea wants to merge 1 commit into
mainfrom
coheigea/remove-OAuthServiceExceptionMapper

Conversation

@coheigea
Copy link
Copy Markdown
Contributor

@reta WDYT about backporting this to 4.1.x as well? I doubt anyone is using it, as it's not referenced anywhere in the codebase

@reta
Copy link
Copy Markdown
Member

reta commented May 20, 2026

@reta WDYT about backporting this to 4.1.x as well? I doubt anyone is using it, as it's not referenced anywhere in the codebase

@coheigea I am somewhat hesitant, this is public class that may be used by applications / services, even if we are not aware of it, deprecation for removal would be a good way to approach it. Thank you

@coheigea coheigea force-pushed the coheigea/remove-OAuthServiceExceptionMapper branch from e90f2a1 to 7d2aa31 Compare May 21, 2026 06:10
@coheigea
Copy link
Copy Markdown
Contributor Author

@reta WDYT about backporting this to 4.1.x as well? I doubt anyone is using it, as it's not referenced anywhere in the codebase

@coheigea I am somewhat hesitant, this is public class that may be used by applications / services, even if we are not aware of it, deprecation for removal would be a good way to approach it. Thank you

Thanks @reta, I'll merge it for main only and mark the class as deprecated for removal on 4.1.x

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants