Hello.
Good to know that the work on this great feature continues!
However this set of patches does not pass `make check-world` on my
laptop.
Here is how I build and test PostgreSQL:
https://github.com/afiskon/pgscripts/blob/master/full-build.sh
Error message:
http://afiskon.ru/s/0b/258c6e4f14_123.txt
regression.diffs:
http://afiskon.ru/s/a0/4993102c32_regression.diffs.txt
regression.out:
http://afiskon.ru/s/4b/acd5a58374_regression.out.txt
Backtrace:
http://afiskon.ru/s/00/c0b32b45a6_bt.txt
I'm using Arch Linux with latest updates, Python version is 3.6.0. I
have no idea what SCRAM has to do with Python, but this issue reproduced
two times of two I did `make check-world`. There is no such issue in
current master branch.
On Mon, Feb 06, 2017 at 02:55:08PM +0200, Heikki Linnakangas wrote:
> I rebased the SCRAM authentication patches over current master. Here you
> are.
>
> I'm trying to whack this into the final shape that it could actually be
> committed. The previous thread on SCRAM authentication has grown
> ridiculously long and meandered into all kinds of details, so I thought it's
> best to start afresh with a new thread.
>
> So, if you haven't paid attention on this for a while, now would be a good
> time to have another look at the patch. I believe all the basic
> functionality, documentation, and tests are there, and there are no known
> bugs. Please review! I'll start reading through these myself again tomorrow.
>
> One thing that's missing, that we need to address before the release, is the
> use of SASLPrep to "normalize" the password. We discussed that in the
> previous thread, and I think we have a good path forward on it. I'd be happy
> to leave that for a follow-up commit, after these other patches have been
> committed, so we can discuss that work separately.
>
> These are also available on Michael's github repository, at
> https://github.com/michaelpq/postgres/tree/scram.
>
> - Heikki
>
--
Best regards,
Aleksander Alekseev
