You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now the scram support does not run the password through sasl prep, so not all technically valid passwords will be able to authenticate using scram. I decided to release scram support even with this deficiency, since without scram support 100% of the passwords for scram would fail ;)
Some information from copied from @jkatz's comments on #176
So what PostgreSQL does is that any UTF8 string goes through SASLprep. If it's not a UTF-8 string, or if it fails at certain parts of the SASLprep, then it just passes the string through. The server-side implementation can be found here:
It doesn't look like Crystal has unicode_normalize_kc or any sort of unicode normalization yet.
Also I'm not sure all of the tables in crystal's stdlib src/unicode/data.cr exactly match up for the things needed for saslprep, so some custom tables might need to be added.
I pushed a branch that mostly just has failing tests, but this seems like it'll be a bit of work before it comes together. So any help here would be welcome.
The text was updated successfully, but these errors were encountered:
Right now the scram support does not run the password through sasl prep, so not all technically valid passwords will be able to authenticate using scram. I decided to release scram support even with this deficiency, since without scram support 100% of the passwords for scram would fail ;)
Some information from copied from @jkatz's comments on #176
and
It doesn't look like Crystal has
unicode_normalize_kc
or any sort of unicode normalization yet.Also I'm not sure all of the tables in crystal's stdlib
src/unicode/data.cr
exactly match up for the things needed for saslprep, so some custom tables might need to be added.I pushed a branch that mostly just has failing tests, but this seems like it'll be a bit of work before it comes together. So any help here would be welcome.
The text was updated successfully, but these errors were encountered: