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

Update Backup info in README.md #171

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Friddrick
Copy link

Hi! I Danilo Soboliev from Gitter, we talk in Docker room about add info to the readme file.

Hi! I Danilo Soboliev from Gitter, we talk in Docker room about add info to the readme file.
Copy link
Collaborator

@ostefano ostefano left a comment

Choose a reason for hiding this comment

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

Thanks for the PR.

I have left some comments.
Also, would it make to specify that for a successful backup you need to backup the files, ssl, .gnupg directories (anything else?), and .env file?

1. **Stop the MISP container**:

```bash
docker compose down>
Copy link
Collaborator

Choose a reason for hiding this comment

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

'>' character needs to be removed

3. **Start the MISP container**:

```bash
docker compose up -d
Copy link
Collaborator

Choose a reason for hiding this comment

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

-d starts them in daemon mode. Since in the readme we never do it, can you delete the -d option?

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