-
Notifications
You must be signed in to change notification settings - Fork 336
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
Upgrade documentation according to the new interface look #7280
Conversation
Co-authored-by: Javier Medeot <[email protected]>
Co-authored-by: Javier Medeot <[email protected]>
Co-authored-by: Javier Medeot <[email protected]>
Co-authored-by: Javier Medeot <[email protected]>
Co-authored-by: Javier Medeot <[email protected]>
Co-authored-by: Javier Medeot <[email protected]>
source/images/cloud-security/github/use-case-github-members-monitoring-alerts-dashboard.png
Outdated
Show resolved
Hide resolved
source/images/cloud-security/github/use-case-github-repository-monitoring-alerts-dashboard.png
Outdated
Show resolved
Hide resolved
source/user-manual/capabilities/file-integrity/interpreting-fim-module-analysis.rst
Outdated
Show resolved
Hide resolved
source/images/getting-started/use-cases/fim/fim-ar-virustotal-alerts.png
Outdated
Show resolved
Hide resolved
source/images/getting-started/use-cases/fim/fim-modified-registry-key-alert-details.png
Outdated
Show resolved
Hide resolved
source/images/getting-started/use-cases/incident-response/ar-module-blocks-dos-attack.png
Outdated
Show resolved
Hide resolved
source/images/getting-started/use-cases/incident-response/ar-module-blocks-dos-attack.png
Outdated
Show resolved
Hide resolved
source/images/getting-started/use-cases/it-hygiene/pci-dss-requirement-violations.png
Outdated
Show resolved
Hide resolved
source/images/getting-started/use-cases/log-data-analysis/alerts-in-index-pattern.png
Outdated
Show resolved
Hide resolved
source/images/getting-started/use-cases/log-data-analysis/cloud-provider-modules.png
Outdated
Show resolved
Hide resolved
source/images/getting-started/use-cases/sca/sca-checks-status-changed.png
Outdated
Show resolved
Hide resolved
source/images/getting-started/use-cases/threat-hunting/security-events.png
Outdated
Show resolved
Hide resolved
source/images/manual/command-monitoring/memory-utilization-alert.png
Outdated
Show resolved
Hide resolved
source/images/manual/managing-agents/endpoints-summary-dashboard.png
Outdated
Show resolved
Hide resolved
source/images/manual/wazuh-indexer/status-evolution-agents-dashboard.png
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The image doesn't show the Save role mapping button. It would be a plus since it's to be clicked in next step and it was visible in previous image. This happens to the similar images that follow. These images appear cropped and look incomplete to my eyes at least.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Improvement suggestion: Higher used memory value and lower available memory value could make the graph standout. Currently the used memory in the graph is little visible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we display the X-axis form in the right as well? Can we use a different time range to add variety to the heat map? Currently we just show two tactics and two techniques only, unlike the previous image.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we could show the Y-axis form on the right as well, it would be a plus.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar to one of the previous comments: A higher memory usage relative to the available memory would improve the memory size metric graph here showing two areas. Currently, yellow area is little visible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Text is about rule ID 92154
and MITRE ID T1053.005
source/images/poc/docker-alerts.png
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PoC is specific to Ubuntu 22 and it's about pulling the NGINX image, running the container, stopping and removing the container. Some alerts are missing (and some others are not relevant).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Step 3 is "Ensure the Ubuntu endpoint is selected". The previous image showed how to ensure the Ubuntu was selected but this is not present in the new image.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: Is it ok to see the syscheck.event
field empty for rule ID 550
alert while we see it with value modified
in the previous image?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previous image with
- Data on the Evolution panel
- masOS endpoint
looked better than this one. The getting started guide is aimed mainly for new users so the nicer the better if possible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This image is cropped at the bottom and graphs look empty
source/user-manual/agent/agent-management/query-configuration.rst
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An additional image showing the specific configuration for the master node like in the previous image could be useful.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line #77 says Stack management > ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing last hyphen. The index pattern name must be my-custom-alerts-*
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
macOS use case uses 10005
check instead
Co-authored-by: Javier Medeot <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
full_log
value doesn't match with document text custom rule definition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Document text says Download CSV in
Line 16 in 3032bbd
To download this report, click on the **Download CSV** beneath the specific property you are interested in on the **Inventory data** page of the endpoint. |
Description
From version 4.8.0, the interface of Wazuh Dashboard had a big change.
This PR aims to replace all the images to match the new interface, as well as reviewing the references to changed sections.
Checks
Docs building
Code formatting and web optimization
redirects.js
script if necessary (check this guide).Writing style
code
font for Bash commands, file names, REST paths, and code.