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

updating docker container default paths for logs,report,data #1545

Merged
merged 1 commit into from
Sep 10, 2019

Conversation

rachid-debu
Copy link
Contributor

@rachid-debu rachid-debu commented Sep 8, 2019

Issue #1494

I didn't edit the part regarding cert.key

or for secured connections (TLS/SSL), please ensure your configuration file
contains the following lines:

    ssl-cert /srv/data/domain.crt
    ssl-key /srv/data/domain.key
    ws-url wss://example.com:8080

I'm not sure about those paths.

-v "/srv/goaccess/data:/goaccess" \
-v "/srv/goaccess/html/index.html:/goaccess/index.html" \
-v "/var/log/apache2/access.log:/goaccess/access.log" \
--name=goaccess allinurl/goaccess \
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This part of the readme used to instruct to run the docker container without specifying any command. For me, this is no longer working, I have to specify a command.

So this part is now a bit redondant with the below part talking about command that can be passed to customize behavior.

@allinurl allinurl merged commit ae7f865 into allinurl:master Sep 10, 2019
@allinurl
Copy link
Owner

Merged. Thank you for submitting this PR!

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

Successfully merging this pull request may close these issues.

2 participants