Skip to content

filebeat: fix typos in crowdstrike, o365, okta, and santa modules#49642

Merged
strawgate merged 2 commits intoelastic:mainfrom
strawgate:strawgate/fix-typos-pr6-security-service-integrations
Mar 24, 2026
Merged

filebeat: fix typos in crowdstrike, o365, okta, and santa modules#49642
strawgate merged 2 commits intoelastic:mainfrom
strawgate:strawgate/fix-typos-pr6-security-service-integrations

Conversation

@strawgate
Copy link
Copy Markdown
Contributor

Summary

Fix spelling and grammar typos in filebeat module documentation and field descriptions.

  • x-pack/filebeat/module/crowdstrike/_meta/docs.md: formatedformatted
  • x-pack/filebeat/module/o365/_meta/docs.md: recomendedrecommended, endpontsendpoints
  • x-pack/filebeat/module/okta/system/_meta/fields.yml: occuredoccurred, informatoninformation (×3), identiferidentifier
  • filebeat/module/santa/_meta/config.yml: remove duplicated word the the
  • Corresponding generated docs and field assets updated

Closes #49082
Closes #49156
Closes partial #49496
Closes partial #49335

Test plan

  • Verify corrected text in each _meta source file
  • Verify generated docs match source fixes

- crowdstrike: 'formated' → 'formatted' in module docs
- o365: 'recomended' → 'recommended', 'endponts' → 'endpoints'
- okta: 'occured' → 'occurred', 'informaton' → 'information' (×3), 'identifer' → 'identifier'
- santa: remove duplicated word 'the the' in config

Closes elastic#49082
Closes elastic#49156
Closes partial elastic#49496
Closes partial elastic#49335

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@strawgate strawgate requested review from a team as code owners March 24, 2026 18:41
@botelastic botelastic Bot added the needs_team Indicates that the issue/PR needs a Team:* label label Mar 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 24, 2026

Warning

Rate limit exceeded

@strawgate has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 0 minutes and 34 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e90fedaf-c68b-4bff-a79d-053783d36d13

📥 Commits

Reviewing files that changed from the base of the PR and between d3f7e4c and 879ccb8.

📒 Files selected for processing (3)
  • docs/reference/filebeat/exported-fields-okta.md
  • docs/reference/filebeat/filebeat-module-crowdstrike.md
  • docs/reference/filebeat/filebeat-module-o365.md
📝 Walkthrough

Walkthrough

This change corrects multiple spelling and wording errors across Filebeat module documentation and field descriptions. The fixes address typos in Okta field documentation (correcting "informaton" to "information," "identifer" to "identifier," and "occured" to "occurred"), CrowdStrike module docs ("formated" to "formatted"), Office 365 module docs ("endponts" to "endpoints" and "recomended" to "recommended"), and Santa module comments (removing duplicated "the the"). A changelog fragment documents the bug fix. Updates are applied consistently across reference documentation files, field definitions in YAML and AsciiDoc formats, module configuration files, and reference YAML templates. No functional code or configuration logic is modified.

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The PR addresses all typo fixes from #49082 (occured→occurred) and #49156 (informaton→information, identifer→identifier) in Okta module metadata and regenerates corresponding documentation. Additional typo fixes in CrowdStrike and O365 modules and Santa module comment duplications are included.
Out of Scope Changes check ✅ Passed All changes are within scope: typo corrections in Filebeat module documentation and metadata files (CrowdStrike, O365, Okta, Santa), plus regenerated docs and field assets. No unrelated functionality modifications.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • 🛠️ Update Documentation: Commit on current branch
  • 🛠️ Update Documentation: Create PR

Comment @coderabbitai help to get the list of available commands and usage tips.

@strawgate strawgate added Team:Security-Service Integrations Security Service Integrations Team backport-skip Skip notification from the automated backport with mergify labels Mar 24, 2026
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

Copy link
Copy Markdown
Contributor

@efd6 efd6 left a comment

Choose a reason for hiding this comment

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

crowdstrike, o365 and okta changes LGTM

(sad that the informatons (atomic units of information?) are going away)

@vishaangelova
Copy link
Copy Markdown
Member

Actually, this needs to be backported to 9.3 for the changes to be published to the docs site (we publish from the latest minor branch).

@vishaangelova vishaangelova added backport-9.3 Automated backport to the 9.3 branch and removed backport-skip Skip notification from the automated backport with mergify labels Mar 24, 2026
Copy link
Copy Markdown
Member

@vishaangelova vishaangelova left a comment

Choose a reason for hiding this comment

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

LGTM!

@strawgate strawgate merged commit 1dd36ff into elastic:main Mar 24, 2026
58 checks passed
mergify Bot pushed a commit that referenced this pull request Mar 24, 2026
…9642)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
(cherry picked from commit 1dd36ff)
vishaangelova added a commit that referenced this pull request Mar 25, 2026
…9642) (#49652)

(cherry picked from commit 1dd36ff)

Co-authored-by: Bill Easton <bill.easton@elastic.co>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Visha Angelova <91186315+vishaangelova@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-9.3 Automated backport to the 9.3 branch Team:Security-Service Integrations Security Service Integrations Team

Projects

None yet

5 participants