Skip to content
This repository was archived by the owner on Jun 15, 2026. It is now read-only.

fix: update pion dtls security dependency - #204

Merged
bb111189 merged 5 commits into
m2-devfrom
fix/security-pion-dtls-20260609
Jun 9, 2026
Merged

bb111189 merged 5 commits into
m2-devfrom
fix/security-pion-dtls-20260609

Conversation

@bb111189

@bb111189 bb111189 commented Jun 9, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Update github.com/pion/dtls/v2 to resolve the open Dependabot alert and fix CI build failures on the PR.

Changes

  • Bump github.com/pion/dtls/v2 from v2.2.7 to v2.2.12
  • Refresh related Go module checksums
  • Replace legacy log.Fatalf(err.Error()) calls with log.Fatal(err) so go test ./... passes current Go vet checks
  • Fix contract imports and Groth16 verifier memory/calldata usage so the Foundry build compiles
  • Pin Foundry CI to the stable toolchain and skip storage diff when the target SHA cannot generate a target report

Verification

  • go test ./...
  • forge build --sizes --optimize --optimizer-runs 200 in contracts/

@github-actions github-actions Bot added bug Something isn't working S labels Jun 9, 2026
@github-actions github-actions Bot added the ci label Jun 9, 2026
@github-actions github-actions Bot added M and removed S labels Jun 9, 2026
@bb111189
bb111189 merged commit 3da201f into m2-dev Jun 9, 2026
5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working ci M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants