search

bwester / consulfs

ConsulFS is a FUSE distributed filesystem backed by a Consul Key-Value store
Other
73 stars 11 forks source link

Ability to support consul token based authentication #8

Closed gshiva closed 3 years ago

gshiva commented 6 years ago

The current code does not support any form of authentication. We want the ability to supply the consul token to consulfs.

Addressed by PR #7

bwester commented 6 years ago

Thanks for contributing! This will be a useful addition.

What do you think about specifying the token in a config file?

gshiva commented 6 years ago

That would work too :)

I just wanted a quick fix :)

On Jan 3, 2018, at 9:51 PM, Benjamin Wester notifications@github.com wrote:

Thanks for contributing! This will be a useful addition.

What do you think about specifying the token in a config file?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

bwester commented 3 years ago

Well, the fix was quick, it just took me 3 years to get to it. My own usage of this tool has dropped off considerably lately. I added a -token-file option to specify a file to read the ACL secret token from. While I was at it, the consulfs command now also supports many of the same environment variables as the Consul agent for configuring the client. So another way to do the same thing would be to set CONSUL_HTTP_TOKEN_FILE. See reference: api.go