Skip to content
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

feat: add private key authentication #109

Merged

Conversation

miloszszymczak
Copy link
Contributor

This adds two additional properties to config allowing to use key pair authentication as described in Key pair authentication

It can be used instead of password and decrypts the key with a passphrase if needed.

@pnadolny13
Copy link
Contributor

@miloszszymczak this looks great, thanks for the PR! The CI tests didnt run since the PR is from a fork but I ran them locally and all passed so there shouldnt be any regressions. I dont have private key auth set up to test it. Do you have it working on your end?

@miloszszymczak
Copy link
Contributor Author

@pnadolny13 yes, I actually made the fork to use it directly in my project and decided later to properly adjust non-code stuff to make a pull request so I can use the official package

@pnadolny13
Copy link
Contributor

pnadolny13 commented Sep 6, 2023

Great! I reviewed the code, all tests pass, and @miloszszymczak confirmed that the changes are working for them. I'll go ahead and merge it!

@pnadolny13 pnadolny13 merged commit 4eb60b8 into MeltanoLabs:main Sep 6, 2023
@miloszszymczak miloszszymczak deleted the feat/support-key-pair-auth branch September 12, 2023 07:50
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.

2 participants