Skip to content

Conversation

@pedosb
Copy link

@pedosb pedosb commented Apr 21, 2025

Breaking change

<placeholder>

Proposed change

It restores the UFW functionality required for #341

Type of change

  • Bugfix (non-breaking change which fixes an issue)
  • New provider (thank you!)
  • Updated provider (thank you!)
  • New feature (which adds functionality to a provider script/repo usage)
  • Breaking change (fix/feature causing existing functionality to break)

Additional information

fixes #341

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

  • Documentation added/updated

@BigRedBrent
Copy link
Contributor

BigRedBrent commented May 27, 2025

I saw your pull request and decided to add its functionality to my pull request, so that it doesn't break any changes that I made: #345

@BigRedBrent
Copy link
Contributor

BigRedBrent commented May 28, 2025

I haven't actually been able to get the firewall to start in my Docker container. Could you test this version of the script to make sure it's properly allowing the ports through the firewall?

https://github.com/haugene/vpn-configs-contrib/raw/d2f2c2e7fcd52f307f17f9e6eb14ea2d824a7ced/openvpn/protonvpn/update-port.sh

Also, what did you have to do to get it to work in your container? I keep getting an error about the Firewall before it even gets to the script that I edited.

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.

Removed UFW updates rules from update-port.sh of protonvpn

2 participants