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.

Sign up for GitHub

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

Jump to bottom

[integrations][zscaler_zia]- Fixed an issue where the "status" attribute was not being parsed in web logs #12297

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[integrations][zscaler_zia]- Fixed an issue where the "status" attribute was not being parsed in web logs #12297

wants to merge 2 commits into from
+194 −3

Conversation

ShourieG
Copy link
Contributor

Type of change

Please label this PR with one of the following labels, depending on the scope of your change:

  • Bug

Proposed commit message

Fixed an issue where the "status" attribute was not being parsed in web logs

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

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@ShourieG ShourieG added integration Label used for meta issues tracking each integration Integration:zscaler_zia Zscaler Internet Access bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations Team [elastic/security-service-integrations] labels Jan 10, 2025
@ShourieG ShourieG self-assigned this Jan 10, 2025
@ShourieG ShourieG requested a review from a team as a code owner January 10, 2025 12:11
@elasticmachine
Copy link

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

kcreddy
kcreddy reviewed Jan 10, 2025
View reviewed changes
Copy link
Contributor

@kcreddy kcreddy left a comment •
edited
Loading

Choose a reason for hiding this comment

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

The attribute status doesn't seem to be part of the Web logs feed and also our current format in the integrations page doesn't include this field.
If this is customised for a particular case, can this be handled inside @custom pipeline?

@elasticmachine
Copy link

elasticmachine commented Jan 10, 2025 •
edited
Loading

💔 Build Failed

Failed CI Steps

History

cc @ShourieG

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

@kcreddy kcreddy kcreddy left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@ShourieG ShourieG

Labels
bugfix Pull request that fixes a bug issue Integration:zscaler_zia Zscaler Internet Access integration Label used for meta issues tracking each integration Team:Security-Service Integrations Security Service Integrations Team [elastic/security-service-integrations]
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ShourieG @elasticmachine @kcreddy