Skip to content

appsec: support for go 1.25#3739

Merged
eliottness merged 1 commit into
mainfrom
eliottness/go-libddwaf-go125
Jul 9, 2025
Merged

appsec: support for go 1.25#3739
eliottness merged 1 commit into
mainfrom
eliottness/go-libddwaf-go125

Conversation

@eliottness

@eliottness eliottness commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

This PR upgrades go-libddwaf to v4.3.0.

Motivation

Unlock go 1.25 for appsec customers

Reviewer's Checklist

  • Changed code has unit tests for its functionality at or near 100% coverage.
  • System-Tests covering this feature have been added and enabled with the va.b.c-dev version tag.
  • There is a benchmark for any new code, or changes to existing code.
  • If this interacts with the agent in a new way, a system test has been added.
  • New code is free of linting errors. You can check this by running golangci-lint run locally.
  • Add an appropriate team label so this PR gets put in the right place for the release notes.
  • Non-trivial go.mod changes, e.g. adding new modules, are reviewed by @DataDog/dd-trace-go-guild.

Unsure? Have a question? Request a review!

@github-actions github-actions Bot added the apm:ecosystem contrib/* related feature requests or bugs label Jul 9, 2025

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@eliottness
eliottness marked this pull request as ready for review July 9, 2025 11:53
@pr-commenter

pr-commenter Bot commented Jul 9, 2025

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2025-07-09 12:05:34

Comparing candidate commit e803875 in PR branch eliottness/go-libddwaf-go125 with baseline commit 802f548 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 24 metrics, 0 unstable metrics.

eliottness commented Jul 9, 2025

Copy link
Copy Markdown
Contributor Author

Merge activity

  • Jul 9, 12:51 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jul 9, 12:51 PM UTC: @eliottness merged this pull request with Graphite.

@eliottness
eliottness merged commit 225b68a into main Jul 9, 2025
243 checks passed
@eliottness
eliottness deleted the eliottness/go-libddwaf-go125 branch July 9, 2025 12:51
eliottness added a commit that referenced this pull request Jul 9, 2025
<!--
* New contributors are highly encouraged to read our
  [CONTRIBUTING](/CONTRIBUTING.md) documentation.
* Commit and PR titles should be prefixed with the general area of the pull request's change.

-->

This PR upgrades go-libddwaf to v4.3.0.

Unlock go 1.25 for appsec customers

<!--
* Authors can use this list as a reference to ensure that there are no problems
  during the review but the signing off is to be done by the reviewer(s).
-->

- [ ] Changed code has unit tests for its functionality at or near 100% coverage.
- [ ] [System-Tests](https://github.com/DataDog/system-tests/) covering this feature have been added and enabled with the va.b.c-dev version tag.
- [ ] There is a benchmark for any new code, or changes to existing code.
- [ ] If this interacts with the agent in a new way, a system test has been added.
- [ ] New code is free of linting errors. You can check this by running `golangci-lint run` locally.
- [ ] Add an appropriate team label so this PR gets put in the right place for the release notes.
- [ ] Non-trivial go.mod changes, e.g. adding new modules, are reviewed by @DataDog/dd-trace-go-guild.

Unsure? Have a question? Request a review!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

apm:ecosystem contrib/* related feature requests or bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants