Skip to content
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

LOG-1500: Linux install script requires password #82

Closed
emptyflash opened this issue Sep 20, 2016 · 2 comments
Closed

LOG-1500: Linux install script requires password #82

emptyflash opened this issue Sep 20, 2016 · 2 comments

Comments

@emptyflash
Copy link

@emptyflash emptyflash commented Sep 20, 2016

Trying to get everything configured to send logs to loggly from within a docker container. It seems that you have to provide your password to the configure linux script for it to work. Is this correct? If so what is the purpose of the token?

I would personally prefer not to have my password stored in my Dockerfile and hosted on a github repo.

@nguyenfilip
Copy link
Contributor

@nguyenfilip nguyenfilip commented Nov 2, 2018

@emptyflash the purpose of the password is twofold:

  1. test that the setup of loggly on the linux machine was successfull
  2. in case AUTH TOKEN is not provided on the commandline, the AUTH token is retrieved using the password from loggly.com

I understand that you don't want to store the password in the Dockerfile, but why you are not passing your secrets (be it password or authtoken) as parameters to your starting container?

@nguyenfilip nguyenfilip changed the title Linux install script requires password LOG-1500: Linux install script requires password Nov 2, 2018
@emptyflash
Copy link
Author

@emptyflash emptyflash commented Nov 14, 2018

I forget the original context of this, but I think the reason I was having issues was because I was trying to run the install script in the build steps of the Dockerfile.

The workaround back then was to run the install script as part of the docker image start up.

These days though, you can specify "build args" that solve this issue.

@emptyflash emptyflash closed this Nov 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.