perara / wg-manager

A easy to use WireGuard dashboard and management tool
MIT License
592 stars 74 forks source link

Adjusted docker compose section #101

Closed flippy1345 closed 3 years ago

flippy1345 commented 3 years ago

I switched the position of ADMIN_USERNAME and ADMIN_PASSWORD. For once it took me 2h to figure out that I changed the username and not the password. For second it is now in the same schema as the other section where a compose file is shown.

vipin-shrivastava commented 3 years ago

@flippy1345

Screenshot_16

After tried with your changes . I got the Exited(1) error.

Can you please tell me the steps to do configuration and also run docker ? Also which docker version are you using?

flippy1345 commented 3 years ago

@vipin-shrivastava image

I adjusted the order of for ADMIN_USERNAME and ADMIN_PASSWORD in the README.md to match the in which order it is written in other section in the README.

I created a stack in portainer and deployed the container. Docker version on my server:

:~$ docker version
Client:
 Version:           20.10.2
 API version:       1.41
 Go version:        go1.13.8
 Git commit:        20.10.2-0ubuntu1~20.04.2
 Built:             Tue Mar 30 21:24:57 2021
 OS/Arch:           linux/amd64
 Context:           default
 Experimental:      true
perara commented 3 years ago

The error cannot be related to this simple change. Looks sensible to have username first then password! Good work!