On 11/22/17 21:08, Michael Paquier wrote: > Yes, agreed. This patch looks good to me. In fe-auth-scram.c, it would > be also nice to add a comment to keep in sync the logics in > build_client_first_message() and build_client_final_message() which > assign the cbind flag value.
Could you clarify what comment you would like to have added or changed? -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
