-
Notifications
You must be signed in to change notification settings - Fork 540
[Imperva] update documentation #17371
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
base: main
Are you sure you want to change the base?
Conversation
|
Pinging @elastic/integration-experience (Team:Integration-Experience) |
Vale Linting ResultsSummary: 17 warnings, 6 suggestions found
|
| File | Line | Rule | Message |
|---|---|---|---|
| packages/imperva/_dev/build/docs/README.md | 18 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/_dev/build/docs/README.md | 65 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/README.md | 18 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/README.md | 65 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/README.md | 207 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/README.md | 220 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/README.md | 242 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'for example' instead of 'e.g'. |
| packages/imperva/docs/README.md | 262 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/knowledge_base/service_info.md | 21 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/knowledge_base/service_info.md | 69 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/knowledge_base/service_info.md | 73 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/knowledge_base/service_info.md | 86 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/knowledge_base/service_info.md | 90 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/knowledge_base/service_info.md | 102 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/knowledge_base/service_info.md | 106 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'. |
| packages/imperva/docs/knowledge_base/service_info.md | 111 | Elastic.EndPuntuaction | Don't end headings with punctuation. |
| packages/imperva/docs/knowledge_base/service_info.md | 154 | Elastic.EndPuntuaction | Don't end headings with punctuation. |
💡 Suggestions (6)
| File | Line | Rule | Message |
|---|---|---|---|
| packages/imperva/_dev/build/docs/README.md | 20 | Elastic.Versions | Use 'or later' instead of 'or higher' when referring to versions. |
| packages/imperva/_dev/build/docs/README.md | 37 | Elastic.WordChoice | Consider using 'refer to (if it's a document), view (if it's a UI element)' instead of 'see', unless the term is in the UI. |
| packages/imperva/_dev/build/docs/README.md | 55 | Elastic.Versions | Use 'or later' instead of 'or higher' when referring to versions. |
| packages/imperva/docs/README.md | 20 | Elastic.Versions | Use 'or later' instead of 'or higher' when referring to versions. |
| packages/imperva/docs/README.md | 37 | Elastic.WordChoice | Consider using 'refer to (if it's a document), view (if it's a UI element)' instead of 'see', unless the term is in the UI. |
| packages/imperva/docs/README.md | 55 | Elastic.Versions | Use 'or later' instead of 'or higher' when referring to versions. |
The Vale linter checks documentation changes against the Elastic Docs style guide.
To use Vale locally or report issues, refer to Elastic style guide for Vale.
🚀 Benchmarks reportTo see the full report comment with |
💚 Build Succeeded
History
|
mjwolf
left a comment
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 looks good to me, it just needs the list in the overview removed
| The Imperva integration for Elastic enables you to collect and analyze logs from Imperva SecureSphere devices using Elastic Agent. By ingesting these logs into the Elastic Stack, you can monitor security events, audit database activity, and gain comprehensive visibility into web application traffic. | ||
|
|
||
| **Securesphere** consists of alerts, violations, and system events. See more details about [alerts, violations, and events](https://docs.imperva.com/bundle/v14.7-web-application-firewall-user-guide/page/1024.htm) | ||
| This integration facilitates: |
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 list can be deleted, it's duplicated by the supported use case section
Proposed commit message
[imperva] Documentation Update
generated new knowledge base file
generated new documentation
Please explain:
This text will be pasted into the squash dialog when the change is committed and will be
a long term historical record of the change to help future contributors understand the
change, please help them by making it clear and comprehensive, they may be you.
If the commit title is adequate to describe both of these things, The text here may be omitted
or replaced with "See title". The title of the PR will be used as the commit message title when
the merge is made and the "See title" marker will be removed if present.
The text here and the PR title will be subject to the PR review process.
-->
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots