Skip to content

[Fortinet Fortiproxy] handle malformed quotes - #15766

Merged
haetamoudi merged 4 commits into
elastic:mainfrom
haetamoudi:6526-fortinet-fortiproxy-malformed-quotes
Oct 29, 2025
Merged

[Fortinet Fortiproxy] handle malformed quotes#15766
haetamoudi merged 4 commits into
elastic:mainfrom
haetamoudi:6526-fortinet-fortiproxy-malformed-quotes

Conversation

@haetamoudi

Copy link
Copy Markdown
Contributor

Proposed commit message

add support for malformed quotes and log version field

Fix for https://github.com/elastic/sdh-beats/issues/6526

  • Add missing log version field logver
  • Remove timestamp field as the event already as a eventime which seems more precise
  • handle malformed quotes

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

@haetamoudi
haetamoudi requested a review from a team as a code owner October 27, 2025 15:48
@haetamoudi haetamoudi added bug Something isn't working, use only for issues documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:fortinet_fortiproxy Fortinet FortiProxy Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience] labels Oct 27, 2025
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/integration-experience (Team:Integration-Experience)

Comment thread packages/fortinet_fortiproxy/data_stream/log/fields/fields.yml Outdated
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

@haetamoudi
haetamoudi merged commit 62cdd1a into elastic:main Oct 29, 2025
7 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

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

@andrewkroh andrewkroh added the bugfix Pull request that fixes a bug issue label Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working, use only for issues bugfix Pull request that fixes a bug issue documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:fortinet_fortiproxy Fortinet FortiProxy Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience]

4 participants