-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Delete member via controller API *doesn't* #859
Comments
@janjaapbos Member deletion was one of the things I was adding to the proxy. But it doesn't seem to work unless this is fixed. |
Does the same thing on my Mac. |
Setting authorized to false and clearing ipAssignments, capabilities, and tags would be a workaround. Since the node will keep trying to join, this is pretty much what a DELETE would accomplish. |
This isn't the same thing if the node was added manually, or has been destroyed. (Meaning that the node is not trying to join.) The DELETE method gives a chance to clean up stale nodes. When you consider the case of ephemeral nodes (which NNG can and does create), the ability to delete becomes even more important. At any rate, the API documents clearly state that DELETE should work. |
Taking a look. Should get fixed in next release. |
So Central seems to lack any API support for this either. It would be nice to have it; treat it as house cleaning. |
I have same behavior on Raspberry Pi using ZeroTier version 1.2.12. |
I can confirm similar behavior on v1.2.12 running on Ubuntu 18.04 |
Bump - can confirm that this is an ongoing issue with version 1.2.12 |
Fixed in |
@adamierymenko It looks like the last release (on the releases tab, not sure if this is accurate) was a year ago. Is there a way to deploy the |
Bump - could this be merged into the public release? |
pretty sure this just shipped |
Wasn't noted in the release notes. |
Umm, in which version? I'm using the latest on repos (1.4.6) and I still get this. |
Looks like it deletes it from the file, but not from memory 😅 |
This will get fixed one day :P |
any update on this issue? Still not working on v1.6.2 |
I have found that this does not work on Windows platform using ZeroTier 1.6.2 in a private network. 1st I remove the member identity as a root from the VL1 configuration. Then I tried all this separate options for VL2 removal:
Now in this case, a subsequent GET actually returns an error as one would expect. The member does not exist. HOWEVER, The only way to work around this is have the member leave the network, which obviously is what we are all trying to get around. Or to create a new network controller and have all the members you want to keep join. It seems the network controller should have more control! |
After discussing this with ZeroTier, the best course of action is to only The DELETE could be used later when a member has been de-authorized for a long period of time. There is no need to restart the service after VL2 changes using the API. |
Wrong DB::get() method being called to look up the network member for deletes
Hi @taniaengel , Sorry for bumping out old thread here. I would like to deauthorize and delete a member via Zerotier localhost:9993 API as mentioned in https://docs.zerotier.com/self-hosting/network-controllers/. Unfortunately, the API unable to remove assgined IP (In this case 10.1.1.200). DELETE Member API Details:
GET Member API Details:
After I've deleted a member, there is no way I could delete the assigned IP anymore. Any solution to solve this issue via API only? So that previously assigned IP could assign to new member ID. Related:
Thanks. |
Deleting a network member using the controller API does not actually delete the resource.
Instead, it looks like it just does the equivalent of a GET.
I would have expected the member to be removed, and the result code to be an HTTP 204 (No Content) result with no JSON body.
Note that this is on a macOS host with the most recent code. API version 4. The member was originally created using a POST to the API.
Best example is curl trace:
Then if you look again, it's still there:
I have not tested this via the "Central API" yet, but that's next.
The text was updated successfully, but these errors were encountered: