-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Go Api for Catalog does not support ACL tokens #1893
Comments
Hi @ddutch take a look at the https://github.com/hashicorp/consul/blob/master/api/api.go#L63-L65 |
I did. But that doesn't work for the catalog. The rest Api documentation only mentions passing a write request object in there body, no parameter. The write options object in Go gets translated to a URL parameter instead of Body. |
Link to the API doc. https://www.consul.io/docs/agent/http/catalog.html#catalog_register |
There is either a bug in the Go api, or in the rest api endpoint. |
Ah sorry about that. I'm thinking it is probably cleaner to teach the REST endpoint about the |
This has been fixed in the register endpoint - #1738. |
The api/catalog.go Register function does not support acl tokens.
The CatalogRegistration struct should likely have a WriteRequest field.
The text was updated successfully, but these errors were encountered: