Skip to content

Update dependencies to latest and bump to go 1.25 - #37

Merged
umputun merged 1 commit into
masterfrom
fix/deps-security-go1.25
Jun 30, 2026
Merged

Update dependencies to latest and bump to go 1.25#37
umputun merged 1 commit into
masterfrom
fix/deps-security-go1.25

Conversation

@paskal

@paskal paskal commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Updates all modules to their latest versions. The patched releases require go 1.25, so the go directive and CI go-version move to 1.25.

Updates

  • slack-go/slack 0.17.3 → 0.27.0 — fixes GHSA-gxhx-2686-5h9g (the Dependabot alert)
  • golang.org/x/net 0.47.0 → 0.56.0 — fixes 7 advisories (GO-2026-5025/5027/5028/5029/5030 html XSS, GO-2026-5026 idna, GO-2026-4918 http2), reached via bluemonday
  • go-pkgz/lgr 0.12.1 → 0.12.3

Notes

  • golang.org/x/net was the more relevant finding here: govulncheck flags it as reachable through bluemonday's HTML sanitiser, whereas the slack advisory is present but not called by this package. Both patched lines require go 1.25.0, which is what forces the toolchain bump.
  • Two gosec G101 findings on test-fixture tokens in telegram_test.go are suppressed with explained //nolint directives.

Verified locally: go test -race, go vet, golangci-lint run, and govulncheck ./... all clean.

Supersedes #36.

Update all modules to their latest versions, which requires go 1.25:
- slack-go/slack 0.17.3 -> 0.27.0 (fixes GHSA-gxhx-2686-5h9g)
- golang.org/x/net 0.47.0 -> 0.56.0 (fixes 7 advisories incl. html XSS)
- go-pkgz/lgr 0.12.1 -> 0.12.3

Bump CI go-version to 1.25 accordingly. Suppress two gosec G101 false
positives on test fixture tokens in telegram_test.go.
@paskal
paskal requested a review from umputun as a code owner June 30, 2026 16:39
@coveralls

coveralls commented Jun 30, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 28460554776

Coverage remained the same at 96.311%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 488
Covered Lines: 470
Line Coverage: 96.31%
Coverage Strength: 8.83 hits per line

💛 - Coveralls

@umputun
umputun merged commit 8c0b20d into master Jun 30, 2026
5 of 6 checks passed
@umputun
umputun deleted the fix/deps-security-go1.25 branch June 30, 2026 17:22
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