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

Update OpenVPN config's SERVER_PUBLIC_URL to UDP in docs #50

Merged
merged 3 commits into from
Mar 27, 2024

Conversation

ay-bh
Copy link

@ay-bh ay-bh commented Mar 12, 2024

Closes #48
This update modifies the OpenVPN configuration documentation to change the SERVER_PUBLIC_URL protocol from TCP to UDP, in order to ensure consistency with the config/config.ini.example file.

@verovaleros
Copy link
Member

We have the same issue here. From our experience with almost a thousand users, our deployment over TCP worked much better than UDP. When using UDP, people experience errors and often, applications not working correctly or not loading.

Let's bring the tcp:// option by default - which is what we recommend, and let's please add the udp:// option commented with 'Switch to this for using OpenVPN over UDP'.

Can you do it @ay-bh ?

@ay-bh
Copy link
Author

ay-bh commented Mar 27, 2024

@verovaleros Thank you for the feedback. Sure I'll do it.

ay-bh added 2 commits March 27, 2024 20:31
Default (recommended) is TCP but the user can uncomment to switch to UDP.
@ay-bh
Copy link
Author

ay-bh commented Mar 27, 2024

@verovaleros Done, I have reverted the changes done in the docs and made suggested changes in the config file. Now everything is consistent.

Copy link
Member

@verovaleros verovaleros left a comment

Choose a reason for hiding this comment

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

Great! 🚀

@verovaleros verovaleros merged commit 86e6b08 into stratosphereips:develop Mar 27, 2024
1 of 2 checks passed
@verovaleros verovaleros added documentation Improvements or additions to documentation enhancement New feature or request labels Mar 27, 2024
@verovaleros verovaleros added this to the v0.1.3-beta milestone Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants