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

Update training-material.md #112

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion training-material.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ The most recent training VM includes:

### TL;DR

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

@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
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

- Training VM system account : `admin/thehive1234`

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
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.

- TheHive URL : http://IP_OF_VM:9000
- Cortex URL : http://IP_OF_VM:9001
- Cortex superAdmin account : `admin/thehive1234`
Expand Down