Skip to content

Absorb pg-pass library #3449

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Absorb pg-pass library #3449

wants to merge 1 commit into from

Conversation

brianc
Copy link
Owner

@brianc brianc commented Apr 29, 2025

To limit supply-chain attack vectors I'm trying to absorb and consolidate all pg-* libraries depended on into this repo so it can be versioned in lock-step and there are no transitive dependencies (except devDependencies) outside of things in this repo. Longer term (I have a TODO comment about it in the code already) I want to remove pg-pass as a dependency and have it something someone can install optionally. It was added before the .password connection param could be made a function for inversion of control. Now that its possible to do whatever you want async to get the password, pg-pass eventually should be not included. In the mean time its a breaking change to remove it and adding pacakges to the monorepo is pretty easy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant