Skip to content
This repository has been archived by the owner on Jul 23, 2024. It is now read-only.

Fix dev secrets backend file path #176

Open
UpGado opened this issue Jul 30, 2020 · 2 comments
Open

Fix dev secrets backend file path #176

UpGado opened this issue Jul 30, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@UpGado
Copy link
Contributor

UpGado commented Jul 30, 2020

It appears that the user that builds the authenticator binaries ends up hardcoded in the path of the dev secrets backend.

To reproduce:

  1. On an EC2 instance with VAULT_ADDR unset.
  2. Run the binary:
ubuntu@ip-172-31-62-204:~$ authenticator --disabletls
INFO   [2020-07-30T07:28:12Z] api starting on http://127.0.0.1:6000
INFO   [2020-07-30T07:28:12Z] loading secrets at "/Users/rebeccapetrin/go/src/github.com/cyralinc/approzium/authenticator/server/testing/secrets.yaml"
ERROR  [2020-07-30T07:28:12Z] authenticator ended due to no valid credential manager available, see debug-level logs for more information
@UpGado UpGado added the bug Something isn't working label Jul 30, 2020
@UpGado UpGado mentioned this issue Jul 31, 2020
@tyrannosaurus-becks
Copy link
Contributor

Hey! I think this might be fixed on the current develop branch. When I make a new authenticator by doing a fresh make dev in approzium/authenticator, it just works for me. Try $ authenticator --dev. If it's still not working for you, can you post fresh steps to reproduce it?

@tyrannosaurus-becks
Copy link
Contributor

Here's an example of it working for me presently:

➜  Desktop authenticator --disable-tls
INFO   [2020-08-07T15:09:36-07:00] api starting on http://127.0.0.1:6000
INFO   [2020-08-07T15:09:36-07:00] loading secrets at "/Users/rebeccapetrin/go/src/github.com/cyralinc/approzium/authenticator/server/testing/secrets.yaml"
INFO   [2020-08-07T15:09:36-07:00] secrets loaded, please restart authenticator to load edits
WARNING[2020-08-07T15:09:36-07:00] local file credential manager should not be used in production
INFO   [2020-08-07T15:09:36-07:00] selected local file as credential manager
WARNING[2020-08-07T15:09:36-07:00] local file credential manager should not be used in production
INFO   [2020-08-07T15:09:36-07:00] grpc starting on http://127.0.0.1:6001
INFO   [2020-08-07T15:09:36-07:00] all ports up and ready to serve traffic

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants