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

fix(privateinternetaccess): fix getting token for port forwarding #1132

Merged
merged 2 commits into from
Sep 5, 2022
Merged

fix(privateinternetaccess): fix getting token for port forwarding #1132

merged 2 commits into from
Sep 5, 2022

Conversation

shoop
Copy link
Contributor

@shoop shoop commented Sep 3, 2022

Connection to PIA using OpenVPN worked fine, but getting a token for the port forwarding service resulted in HTTP errors.
Looking at the latest official shell script to get a token it seems that the token URL and method have changed. This PR adjusts gluetun to match.

Note: this is my first time writing/using Go.

qdm12
qdm12 previously approved these changes Sep 4, 2022
Copy link
Owner

@qdm12 qdm12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome thanks for fixing/sharing this!

How was your development setup journey? Did you find the development setup / contributing page in the wiki or did you go all in without documentation?

@qdm12 qdm12 merged commit 532df9f into qdm12:master Sep 5, 2022
@shoop
Copy link
Contributor Author

shoop commented Sep 5, 2022

Awesome thanks for fixing/sharing this!

How was your development setup journey? Did you find the development setup / contributing page in the wiki or did you go all in without documentation?

Hi @qdm12 , no problem! You beat me to the punch here, thanks for fixing it up.
I did find that page and it definitely helped. I got sidetracked a bit by trying to use the Fedora packaged go version which was a mistake but nothing that a toolbox devcontainer + manual install could not fix.

Once again, thanks for creating and maintaining gluetun!

@qdm12
Copy link
Owner

qdm12 commented Sep 5, 2022

Awesome thanks! Asking this since I'm recording myself doing gluetun development (setup environment, add a vpn provider). Thanks again for the fix, I'll do a release soon-ish.

CMarcJoubert pushed a commit to FideresDev/gluetun that referenced this pull request Jan 14, 2025
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