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.

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

feat: SASL client support #624

Open
rkrishn7 opened this issue Oct 17, 2023 · 2 comments · May be fixed by #631
Open

feat: SASL client support #624

rkrishn7 opened this issue Oct 17, 2023 · 2 comments · May be fixed by #631

Comments

@rkrishn7
Copy link

Is your feature request related to a problem? Please describe.

Currently, there is no client support for SASL authentication when connecting to the proxy

Describe the solution you'd like

Add SASL authentication as a configurable strategy. The default authentication mechanism will remain md5.

This will likely involve refactoring the authentication logic, possibly into a separate module.

@rkrishn7
Copy link
Author

I'm happy to work on this!

@WagnerPMC
Copy link

WagnerPMC commented Oct 19, 2023

Is your feature request related to a problem? Please describe.

Currently, there is no client support for SASL authentication when connecting to the proxy

Describe the solution you'd like

Add SASL authentication as a configurable strategy. The default authentication mechanism will remain md5.

This will likely involve refactoring the authentication logic, possibly into a separate module.

I've been doing this for the last two weeks. I don't have a lot of time, but I've already made a lot of progress.
I plan to rewrite almost entirely SCRAM authentication as it is now, and add client-server SCRAM authentication.
I think by the end of the month I'll have a result and as a consequence a Pull Request.

p.s. I'm not a PGCat contributor, so PR decisions are out of my hands.

@rkrishn7 rkrishn7 linked a pull request Oct 30, 2023 that will close this issue
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 a pull request may close this issue.

2 participants