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

Adds documentation for log type categories #5181

Merged
merged 19 commits into from
Oct 16, 2023
Merged

Adds documentation for log type categories #5181

merged 19 commits into from
Oct 16, 2023

Conversation

hdhalter
Copy link
Contributor

Description

Categories have been added to log types for easier filtering and sorting.

Issues Resolved

Fixes #5147

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Heather Halter <[email protected]>
@hdhalter hdhalter self-assigned this Oct 10, 2023
@hdhalter hdhalter added 3 - Tech review PR: Tech review in progress release-notes PR: Include this PR in the automated release notes v2.11.0 labels Oct 10, 2023
@hdhalter
Copy link
Contributor Author

To help with the tech/doc review, here are some of the changes:

On Supported log types topic:

  • In the first paragraph I added the info about the new category feature
  • The second paragraph was taken from the 'Create custom log detector' page. I just added the bit about the Category column
  • New column for 'Category' in the table
  • New Security > Waf log type description
  • Page actions was taken from 'Create detector' topic, so the only new text is the last bullet about the category.

In the 'Create custom..' topic:

  • Much of the content was moved, as noted above, to the other page where it was more relevant.
  • Simplified the steps and added the part about selecting a category.
  • I also added links back to the log types page.

Signed-off-by: Heather Halter <[email protected]>
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
Signed-off-by: Heather Halter <[email protected]>
@hdhalter
Copy link
Contributor Author

I've added a new column called "Log names" with the full names. Can someone please provide more detailed descriptions of the log types? Thanks.

minor changes needed in the names would be Microsoft Azure for Azure,  Linux System logs instead of Sys logs

Signed-off-by: Heather Halter <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
@hdhalter hdhalter added 4 - Doc review PR: Doc review in progress and removed 3 - Tech review PR: Tech review in progress labels Oct 11, 2023
Signed-off-by: Heather Halter <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
@hdhalter
Copy link
Contributor Author

@jimishs - I removed the Log Name column. Can you please approve? Thanks.

Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

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

LGTM with comments

_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
@hdhalter hdhalter added 5 - Editorial review PR: Editorial review in progress and removed 4 - Doc review PR: Doc review in progress labels Oct 12, 2023
Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

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

@hdhalter Please see my comments and changes and let me know if you have any questions. Thanks!

_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
| Network Activity| `Dns` | DNS logs that store Domain Name System (DNS) activity. |
| Network Activity | `Network` | Network logs that record events that happen in a system's network, such as login attempts and application events. |
| Network Activity | `vpcflow` | [VPC Flow Logs](https://docs.aws.amazon.com/prescriptive-guidance/latest/logging-monitoring-for-application-owners/vpc-flow-logs.html) that capture information about the IP traffic going to and from network interfaces in your VPC. |
| Security | `Waf` | Web Application Firewall (WAF) logs (introduced in OpenSearch 2.11) for users that require monitoring of the WAF use case that's provided out-of-the box in the Security plugin. The role of WAF is to monitor and filter HTTP traffic between a web application and the internet. WAF prevents common security attacks, such as cross-site scripting (XSS), SQL Injection (SQi), and so on. |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are we referring to AWS WAF here? If so, the service name must always be "AWS WAF", never shortened to "WAF", and we don't define WAF as "Web Application Firewall". Instead, we say "AWS WAF is a web application firewall."

_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
_security-analytics/sec-analytics-config/log-types.md Outdated Show resolved Hide resolved
hdhalter and others added 4 commits October 13, 2023 14:45
Editorial updates.

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Fixed case in table.

Signed-off-by: Heather Halter <[email protected]>
Double-checked the UI and category names are capitalized.

Signed-off-by: Heather Halter <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
@hdhalter
Copy link
Contributor Author

@natebower - I've accepted your revisions. Thanks! I am just waiting on confirmation from development about AWS WAF.

@hdhalter hdhalter removed the 5 - Editorial review PR: Editorial review in progress label Oct 16, 2023
@hdhalter hdhalter merged commit b8c53f6 into main Oct 16, 2023
harshavamsi pushed a commit to harshavamsi/documentation-website that referenced this pull request Oct 31, 2023
* new log categories

Signed-off-by: Heather Halter <[email protected]>

* fixed topic name in link

Signed-off-by: Heather Halter <[email protected]>

* added log names to table

Signed-off-by: Heather Halter <[email protected]>

* Update log-types.md

minor changes needed in the names would be Microsoft Azure for Azure,  Linux System logs instead of Sys logs

Signed-off-by: Heather Halter <[email protected]>

* Update log-types.md

Signed-off-by: Heather Halter <[email protected]>

* remove log name column

Signed-off-by: Heather Halter <[email protected]>

* remove table column formatting

Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/custom-log-type.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Apply suggestions from code review

Editorial updates.

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update log-types.md

Fixed case in table.

Signed-off-by: Heather Halter <[email protected]>

* Update log-types.md

Double-checked the UI and category names are capitalized.

Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

---------

Signed-off-by: Heather Halter <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
vagimeli pushed a commit that referenced this pull request Dec 21, 2023
* new log categories

Signed-off-by: Heather Halter <[email protected]>

* fixed topic name in link

Signed-off-by: Heather Halter <[email protected]>

* added log names to table

Signed-off-by: Heather Halter <[email protected]>

* Update log-types.md

minor changes needed in the names would be Microsoft Azure for Azure,  Linux System logs instead of Sys logs

Signed-off-by: Heather Halter <[email protected]>

* Update log-types.md

Signed-off-by: Heather Halter <[email protected]>

* remove log name column

Signed-off-by: Heather Halter <[email protected]>

* remove table column formatting

Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/custom-log-type.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Apply suggestions from code review

Editorial updates.

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

* Update log-types.md

Fixed case in table.

Signed-off-by: Heather Halter <[email protected]>

* Update log-types.md

Double-checked the UI and category names are capitalized.

Signed-off-by: Heather Halter <[email protected]>

* Update _security-analytics/sec-analytics-config/log-types.md

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Heather Halter <[email protected]>

---------

Signed-off-by: Heather Halter <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
@hdhalter hdhalter deleted the log-types-211 branch March 28, 2024 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-notes PR: Include this PR in the automated release notes v2.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] Log type categories
6 participants