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

Peer does not get removed from server config if removed from GUI. #22

Closed
sputnic42 opened this issue Jul 14, 2020 · 1 comment
Closed
Assignees
Labels
bug Something isn't working
Projects

Comments

@sputnic42
Copy link

If a peer is created on a sever and then deleted it still remains in the server configuration an is still able to connect.
If a new peer is created on the same server both peers are visible in server config and able to connect.

The old deleted peer dissappears from server configuration after server restart, but only if a new peer was created afterwards.

Steps to reproduce:

  1. create server
  2. create peer1
    2.1 connect witch peer1
  3. delete peer1
    3.1 connect with peer1 (should still work)
  4. create new peer2
    4.1 both peers sould be able to connect
  5. restart server
    5.1 peer1 sould now not be able to connect anymore and be gone fom server config
@perara perara added the bug Something isn't working label Jul 19, 2020
@perara perara self-assigned this Jul 19, 2020
@perara perara added this to In progress in wg-manager Jul 19, 2020
perara added a commit that referenced this issue Jul 19, 2020
* Fixed bug where server interface were queried using ID and not the name. Fixes #22
@perara
Copy link
Owner

perara commented Jul 19, 2020

Hi, Thanks for the bug report.
I found a bug related to this. The deletion code tried to lookup the interface by ID and not the actual interface name. Additionally, I've suppressed exceptions where the interface is missing. Should be fixed now, and I've made a logging message when this happens.

Please reopen if the problem persists (pull from dev)

@perara perara closed this as completed Jul 19, 2020
perara added a commit that referenced this issue Jul 19, 2020
@perara perara moved this from In progress to Done in wg-manager Jul 19, 2020
m4xx1m added a commit to m4xx1m/wg-manager that referenced this issue Oct 16, 2022
* Fixed bug where server interface were queried using ID and not the name. Fixes perara#22
m4xx1m added a commit to m4xx1m/wg-manager that referenced this issue Oct 16, 2022
m4xx1m pushed a commit to m4xx1m/wg-manager that referenced this issue Oct 16, 2022
* Fixed bug where server interface were queried using ID and not the name. Fixes perara#22
m4xx1m pushed a commit to m4xx1m/wg-manager that referenced this issue Oct 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
wg-manager
  
Done
Development

No branches or pull requests

2 participants