-
Notifications
You must be signed in to change notification settings - Fork 17
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
#1656 Split up scram_over_mtls module into submodules #1679
#1656 Split up scram_over_mtls module into submodules #1679
Conversation
CodSpeed Performance ReportMerging #1679 will create unknown performance changesComparing Summary
Benchmarks breakdown
|
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.
I pasted the moved code into a diff tool and can confirm that the code is unchanged, except for a few error message typo fixes.
Nice!
Thanks for checking! JOOC, what diff tool did you use? |
Not sure 🤣 |
From my history it was: https://www.diffchecker.com/ |
This PR splits up the original monolithic module
scram_over_mtls
into submodulesrecreate_token_queue
andcreate_token
in order to prepare for the improvement on token handling in #1667. It doesn't change the functionality of the modulescram_over_mtls
.Closes #1656