Skip to content
This repository was archived by the owner on Jul 27, 2023. It is now read-only.

Introduced acl and auth support #122

Merged
merged 3 commits into from
May 4, 2016

Conversation

YannRobert
Copy link
Contributor

basic support for the Consul ACL (providing token) + support for Basic Authentication (providing username+password)

Added support for the Consul ACL : a token can be provided to the Consul.Builder, this token will be added to every request as a query parameter.

Added support for Basic Authentication : in case you setup a reverse proxy in front of the Consul service (for additional security to the ACL, or as an alternative to the Consul ACL)

In both case, an Interceptor is added to the Retrofit client.

YannRobert added 3 commits May 4, 2016 11:24

Verified

This commit was signed with the committer’s verified signature.
jrjohnson Jon Johnson
in order to allow the creation of a Consul instance, without 'pinging' the server with AgentClient

This allows the creation of a Consul instance while the Consul service is unavailable.
This is useful when the Consul instance is created at application startup (Spring Framework style), and we want the application to be able to start (not fail) even when Consul is not available.

Verified

This commit was signed with the committer’s verified signature.
jrjohnson Jon Johnson
…c Authentication (providing username+password)

Added support for the Consul ACL : a token can be provided to the Consul.Builder, this token will be added to every request as a query parameter.

Added support for Basic Authentication : in case you setup a reverse proxy in front of the Consul service (for additional security to the ACL, or as an alternative to the Consul ACL)

In both case, an Interceptor is added to the Retrofit client.

Verified

This commit was signed with the committer’s verified signature.
jrjohnson Jon Johnson
…the project builds with Java 7
@rickfast
Copy link
Owner

rickfast commented May 4, 2016

Awesome thanks!

@rickfast rickfast merged commit 342b24b into rickfast:master May 4, 2016
@YannRobert
Copy link
Contributor Author

Thank you for merging so fast.
Would be possible to release ?

@rickfast
Copy link
Owner

rickfast commented May 4, 2016

Sure. Once I'm in the office

@rickfast
Copy link
Owner

rickfast commented May 4, 2016

0.11.2 is available

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants