Skip to content

[Beyondtrust PRA] Change session_owner type from keyword to boolean and Update documentation#18565

Merged
moxarth-rathod merged 2 commits into
elastic:mainfrom
moxarth-rathod:beyondtrust-bugfixes
May 6, 2026
Merged

[Beyondtrust PRA] Change session_owner type from keyword to boolean and Update documentation#18565
moxarth-rathod merged 2 commits into
elastic:mainfrom
moxarth-rathod:beyondtrust-bugfixes

Conversation

@moxarth-rathod
Copy link
Copy Markdown
Contributor

@moxarth-rathod moxarth-rathod commented Apr 22, 2026

Proposed commit message

beyondtrust_pra: change session_owner type from keyword to boolean

It was incorrectly mapped as keyword.

Also, remove GCP from the documentation, this integration uses CEL-only.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

  • Clone integrations repo.
  • Install elastic package locally.
  • Start elastic stack using elastic-package.
  • Move to integrations/packages/beyondtrust_pra directory.
  • Run the following command to run tests.

elastic-package test

Related issues

@moxarth-rathod moxarth-rathod self-assigned this Apr 22, 2026
@moxarth-rathod moxarth-rathod added breaking change bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Integration:beyondtrust_pra BeyondTrust PRA labels Apr 22, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 22, 2026

Vale Linting Results

Summary: 2 warnings, 1 suggestion found

⚠️ Warnings (2)
File Line Rule Message
packages/beyondtrust_pra/_dev/build/docs/README.md 27 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'.
packages/beyondtrust_pra/docs/README.md 27 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'.
💡 Suggestions (1)
File Line Rule Message
packages/beyondtrust_pra/docs/README.md 103 Elastic.WordChoice Consider using 'can, might' instead of 'may', unless the term is in the UI.

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.

Comment thread packages/beyondtrust_pra/changelog.yml Outdated
@moxarth-rathod moxarth-rathod marked this pull request as ready for review April 22, 2026 05:56
@moxarth-rathod moxarth-rathod requested a review from a team as a code owner April 22, 2026 05:56
@elasticmachine
Copy link
Copy Markdown

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

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Apr 22, 2026
@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

cc @moxarth-rathod

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.

Is there a backing issue for this?

@moxarth-rathod
Copy link
Copy Markdown
Contributor Author

Is there a backing issue for this?

Created an issue: #18677

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.

Note change in proposed commit message.

Also, in looking at this, the initial json processor has an on_failure processor. I think that it should probably not since if it fails, nothing further will be successful. Not for this PR.

/cc @kcreddy

@moxarth-rathod moxarth-rathod merged commit 47f27de into elastic:main May 6, 2026
12 checks passed
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

Package beyondtrust_pra - 0.4.0 containing this change is available at https://epr.elastic.co/package/beyondtrust_pra/0.4.0/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change bugfix Pull request that fixes a bug issue documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:beyondtrust_pra BeyondTrust PRA Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Beyondtrust PRA] fix session_owner type and update documentation

4 participants