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

[BUG] Setting SERVER_ALLOWEDIPS_PEER_ does not recreate configs #342

Closed
1 task done
rwrnet opened this issue Jul 10, 2024 · 2 comments
Closed
1 task done

[BUG] Setting SERVER_ALLOWEDIPS_PEER_ does not recreate configs #342

rwrnet opened this issue Jul 10, 2024 · 2 comments

Comments

@rwrnet
Copy link

rwrnet commented Jul 10, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

If you set the ENV SERVER_ALLOWEDIPS_PEER_... peer configs are not recreated.

Log also states: **** No changes to parameters. Existing configs are used. ****

This is due to:

if [[ "$SERVERURL" != "$ORIG_SERVERURL" ]] || [[ "$SERVERPORT" != "$ORIG_SERVERPORT" ]] || [[ "$PEERDNS" != "$ORIG_PEERDNS" ]] || [[ "$PEERS" != "$ORIG_PEERS" ]] || [[ "$INTERFACE" != "$ORIG_INTERFACE" ]] || [[ "$ALLOWEDIPS" != "$ORIG_ALLOWEDIPS" ]] || [[ "$PERSISTENTKEEPALIVE_PEERS" != "$ORIG_PERSISTENTKEEPALIVE_PEERS" ]]; then
not checking for changes on the SERVER_ALLOWEDIPS_PEER Envs.

Expected Behavior

If one adds a SERVER_ALLOWEDIPS_PEER_... config is being recreated.

Steps To Reproduce

  1. start a fresh server config
  2. add SERVER_ALLOWEDIPS_PEER_... for one of the peers
  3. recreate pod
  4. check for allowedip config added to wg0.conf

Environment

- OS: Ubuntu 24.04
- How docker service was installed: via apt install docker-ce

CPU architecture

x86-64

Docker creation

It should be reproducible with every config. Happy to provide my anonymized specific one, in case it is needed.

Container logs

**** Server mode is selected ****
**** No changes to parameters. Existing configs are used. ****
Copy link

Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid.

@rwrnet
Copy link
Author

rwrnet commented Jul 10, 2024

Just realized, that the documentation states this explicitely. Closing issue.

@rwrnet rwrnet closed this as completed Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

1 participant