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

Various improvement #216

Draft
wants to merge 10 commits into
base: v3.1-dev
Choose a base branch
from
Draft

Various improvement #216

wants to merge 10 commits into from

Conversation

kuhnchris
Copy link

Hi!

I took the liberty to adapt some things on the dashboard side to support various little things:

  • fixed getPeerDataUsage in api.py --> removed the hardcoded "wg0"
  • added a Pipfile (for pipenv)
  • added a CLI with a couple of comfort functions (get peer config, get peer config qr, get configs)
  • Added "-" as illegal sign, as it causes conflicts
  • removed some print statements
  • Added the possibility of using SCRIPT_ROOT to actually mount the application in a different sub-url (for example
    environment:
      - SCRIPT_NAME=/wg

causes the app be reachable within the /wg sub url)

  • Increased the error correction level on the qr code to 'H' (-> 30%), as it was not readable by the wireguard android app
  • changed tons of hardcoded URLs to use {{url_for}} which respects SCRIPT_ROOT
  • Added optional supervisord support for docker
  • Added max interface length at 15 chars, as any longer it's not working.

Feel free to pick and choose the changes you may want to take over, I keep this as a draft, let me know if/how you want to proceed.

Thanks!

@donaldzou
Copy link
Owner

Hi! Thank you so much for working on the improvement and I hope you like this project. I would love to look into it but due to I'm trying to be work life balanced, I decided to pause the development on this project, until I go back to school soon. I hope the best on your coding journey and might talk to you soon ;)

@kuhnchris
Copy link
Author

No problem at all, as I mentioned, feel free to take over whatever you need if you ever find the time to work on it again, I'm currently using it in an internal deployment for gatekeeping services, so I may add some more functions if I need them, just drop me a heads-up if you ever feel like continuing on this, then I'll try to clean up what I did the best I can. ;-)

Good luck with your studies and stay strong!

@seejm
Copy link

seejm commented Mar 31, 2023

Hi! Thank you so much for working on the improvement and I hope you like this project. I would love to look into it but due to I'm trying to be work life balanced, I decided to pause the development on this project, until I go back to school soon. I hope the best on your coding journey and might talk to you soon ;)

Can i get a access your updates . i am running 3.1-dev now

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.

None yet

3 participants