sqalx icon indicating copy to clipboard operation
sqalx copied to clipboard
verified publisher icon heetch

Allow "pgx" driver to be used with SavePoint

Open sammy-tam opened this issue 6 years ago • 1 comments

Currently the SAVEPOINT function checks for driver name == "postgres". Since "pgx" driver is now supported by sqlx as well, it'd be great and we allow both.

sammy-tam avatar Aug 16 '19 01:08 sammy-tam

Hello (and sorry for the extra late reply!) This is definitely a change we can include

Would you care to open a pull request that includes tests? Thanks in advance

skateinmars avatar Feb 23 '22 12:02 skateinmars

Hi @skateinmars ! Just made a PR to fix this issue earlier today. Would you mind reviewing/merging it, although it's not a security fix? 🙏 We use sqalx extensively and just started using nested transactions, so this change really would go a long way for us!

Thiht avatar Jul 11 '23 13:07 Thiht