-
Notifications
You must be signed in to change notification settings - Fork 553
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
Adding: Wireguard VPN #1003
base: master
Are you sure you want to change the base?
Adding: Wireguard VPN #1003
Conversation
Testing Note: Testing this pull request might be a bit tricky due to the nature of the WireGuard VPN service and my current setup. Here are the details:
Given these constraints, I've done my best to ensure that the changes adhere to best practices and have thoroughly reviewed the code. However, I understand the importance of proper testing. If any other contributors or reviewers have suggestions for overcoming these testing challenges or have specific testing instructions, please feel free to share them. I appreciate your understanding and cooperation in this matter. Please let me know if you have any questions or need further clarification regarding the changes in this pull request. |
Yes, overall it looks good to me as well! Probably someone can rest and once confirmed to be working, I'm happy to merge! I don't know wiregaurd workings and what to test, so unfortunately I'm unable to test. Alternatively, you can ping me on Slack ( |
OK thanks. I am going to try and find a solution to test the VPN. I'll ping you if needed. FYI here is a list of what I will be testing:
|
@Duckiduc Have you made any progress on this? |
Not at all. I've been quite busy the last few weeks... didn't have time to properly test it. |
Great work @Duckiduc, is there an update on this or anything we can support with? |
Hey @brenner-tobias! Thanks :) Actually yes I have tested it and it works fine but I won't have the time to maintain it... |
@Duckiduc EDIT: I have been able to turn on HTTPS by turning on the WebSockets option first... Still no idea why this works but well! |
Hi! Glad that you managed to make it work. Thanks for the feedback I'll add that to the documentation. |
CAP_ADD is now added to caprover master branch. It should be officially release soon. |
I did install using TEMPLATE in one click apps Thanks a lot. |
☑️ Self Check before Merge
npm ci && npm run validate_apps && npm run formatter