brianc
Kerberos support for node-postgres
@brianc We'd ideally like to get this merged and released through the main package. Could you let us know what the procedure would be? There's unfortunately not a lot of documentation for this flow, and it's not well trodden.
We'd ideally like to get this merged and released through the main package. Could you let us know what the procedure would be?
I can do that! Its very easy on my side to release new versions. Ideally all features should have tests along side them. I realize testing against a kerberos system is.... unpleasant ... do you have any way you could include tests? Even the pg-protocol stuff typically ships with unit tests. Its very high likelyhood without tests sometime within the next 10 years this code will change and/or break without them.
We'd ideally like to get this merged and released through the main package. Could you let us know what the procedure would be?
I can do that! Its very easy on my side to release new versions. Ideally all features should have tests along side them. I realize testing against a kerberos system is.... unpleasant ... do you have any way you could include tests? Even the
pg-protocolstuff typically ships with unit tests. Its very high likelyhood without tests sometime within the next 10 years this code will change and/or break without them.
Cool, I'll try to take a stab at writing some tests for this change and circle back afterwards. Thanks!
Also, is CI not configured to run automatically on changes to the branch?
Also, is CI not configured to run automatically on changes to the branch?
Boo i need to set that up. having to keep re-approving it is no bueno!