Skip to content

[cisco_secure_email_gateway] Fix parsing the text_mail log lines for file uploads#11815

Merged
aleksmaus merged 0 commit intoelastic:mainfrom
aleksmaus:fix/cisco_secure_email_gateway_5392
Nov 22, 2024
Merged

[cisco_secure_email_gateway] Fix parsing the text_mail log lines for file uploads#11815
aleksmaus merged 0 commit intoelastic:mainfrom
aleksmaus:fix/cisco_secure_email_gateway_5392

Conversation

@aleksmaus
Copy link
Contributor

Proposed commit message

[cisco_secure_email_gateway] Fix parsing the text_mail log lines for file uploads

Some log lines where unaccounted for and not properly parsed.

  • Added new patters to match
  • Added the ECS field mapping for file.hash.sha256

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

@aleksmaus aleksmaus added Integration:cisco_secure_email_gateway Cisco Secure Email Gateway bugfix Pull request that fixes a bug issue Team:Security-Deployment and Devices DEPRECATED Deployment and Devices Security team [elastic/sec-deployment-and-devices] labels Nov 21, 2024
@aleksmaus aleksmaus self-assigned this Nov 21, 2024
@aleksmaus aleksmaus requested a review from a team as a code owner November 21, 2024 16:48
@elasticmachine
Copy link

Pinging @elastic/sec-deployment-and-devices (Team:Security-Deployment and Devices)

@paulrossmeier
Copy link

Looks like the changelog version needs to be bumped

@aleksmaus
Copy link
Contributor Author

Looks like the changelog version needs to be bumped

Yep, absolutely. Typo. Fixed now.

@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elastic-sonarqube
Copy link

@elasticmachine
Copy link

💚 Build Succeeded

History

  • 💔 Build #18578 failed b2802df2c3caef84d12469a9eafe919adbd93ea0
  • 💔 Build #18565 failed b9559a35dbaf09121a68a8f82370b844de0bb56f

cc @aleksmaus

Copy link
Contributor

@mjwolf mjwolf left a comment

Choose a reason for hiding this comment

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

👍

@aleksmaus aleksmaus merged commit 75bfde7 into elastic:main Nov 22, 2024
@elastic-vault-github-plugin-prod

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

qcorporation pushed a commit that referenced this pull request Feb 3, 2025
…file uploads (#11815)

* [cisco_secure_email_gateway] Fix parsing the text_mail log lines for file uploads

* Update changelog with PR number

* Fix changelog

* Update README.md
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
…file uploads (elastic#11815)

* [cisco_secure_email_gateway] Fix parsing the text_mail log lines for file uploads

* Update changelog with PR number

* Fix changelog

* Update README.md
qcorporation pushed a commit that referenced this pull request Feb 4, 2025
…file uploads (#11815)

* [cisco_secure_email_gateway] Fix parsing the text_mail log lines for file uploads

* Update changelog with PR number

* Fix changelog

* Update README.md
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
…file uploads (elastic#11815)

* [cisco_secure_email_gateway] Fix parsing the text_mail log lines for file uploads

* Update changelog with PR number

* Fix changelog

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

Labels

bugfix Pull request that fixes a bug issue Integration:cisco_secure_email_gateway Cisco Secure Email Gateway Team:Security-Deployment and Devices DEPRECATED Deployment and Devices Security team [elastic/sec-deployment-and-devices]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants