Skip to content
This repository was archived by the owner on Sep 20, 2023. It is now read-only.

Update training-material.md#112

Open
benny132 wants to merge 1 commit intoTheHive-Project:masterfrom
benny132:patch-1
Open

Update training-material.md#112
benny132 wants to merge 1 commit intoTheHive-Project:masterfrom
benny132:patch-1

Conversation

@benny132
Copy link
Copy Markdown

On the latest VM download there is no user called thehive, it's just an admin account

On the latest VM download there is no user called thehive, it's just an admin account
@scotmatson
Copy link
Copy Markdown

The current VM for TheHive uses both thehive and admin, note that verbiage referencing the system account not the web application.

If anything an additional reference to the web application credentials should be present.


### TL;DR

- Training VM system account : `thehive/thehive1234`
Copy link
Copy Markdown

@scotmatson scotmatson Sep 5, 2019

Choose a reason for hiding this comment

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

Keep this line. It is a reference to the VM itself, not the web application.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this is why the point is named Training VM system account

### TL;DR

- Training VM system account : `thehive/thehive1234`
- Training VM system account : `admin/thehive1234`
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Move this line to precede the current line 19 with the reference to Cortex.

Text should read, "TheHive superAdmin account : admin/thehive1234

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

there is a difference btw system and application accounts.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants