Skip to content

Feature/parser envoy k8s#1622

Merged
LaurenceJJones merged 13 commits intocrowdsecurity:masterfrom
Yanis-Kouidri:feature/parser-envoy-k8s
Jan 13, 2026
Merged

Feature/parser envoy k8s#1622
LaurenceJJones merged 13 commits intocrowdsecurity:masterfrom
Yanis-Kouidri:feature/parser-envoy-k8s

Conversation

@Yanis-Kouidri
Copy link
Copy Markdown
Contributor

Description

New Parser: Envoy JSON logs for Kubernetes (CRI).
This parser extracts HTTP metadata from Envoy Gateway logs and passes them to the next stage (http-logs).
Then it can be use for scenarios like the ones in base-http-scenarios collection

Checklist

  • I have read the contributing guide
  • I have tested my changes locally
  • For new parsers or scenarios, tests have been added
  • I have run the hub linter and no issues were reported (see contributing guide)
  • Automated tests are passing
  • AI was used to generate any/all content of this PR

@LaurenceJJones LaurenceJJones self-requested a review January 10, 2026 19:08
Copy link
Copy Markdown
Contributor

@LaurenceJJones LaurenceJJones left a comment

Choose a reason for hiding this comment

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

lgtm @Yanis-Kouidri waiting for your 👍🏻 on my changes

@LaurenceJJones LaurenceJJones force-pushed the feature/parser-envoy-k8s branch from 68ba374 to 20c7883 Compare January 12, 2026 10:49
Copy link
Copy Markdown
Contributor Author

@Yanis-Kouidri Yanis-Kouidri left a comment

Choose a reason for hiding this comment

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

Good for me

@LaurenceJJones LaurenceJJones merged commit a6b10f3 into crowdsecurity:master Jan 13, 2026
3 checks passed
- parsed: raw_remote_addr
expression: evt.Unmarshaled.envoy.downstream_remote_address
- parsed: remote_addr
expression: "evt.Unmarshaled.envoy.downstream_remote_address != nil ? Split(evt.Unmarshaled.envoy.downstream_remote_address, ':')[0] : nil"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey @Yanis-Kouidri, thank you for your work on this! I'm deploying it in a dual-stack cluster and I think that this is causing it to fail to parse IPv6 addresses, since it only returns the first hext:

failed to parse source_ip [2001: ParseAddr(\"[2001\"): unable to parse IP

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you open an issue with an example ipv6 line so we can add some tests

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sure! I've opened #1658

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hey @harryjph, well spotted, I didn't mind to IPv6 address, thanks for your contribution !

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants