Skip to content

Go 1_21 Upgrade + Dependency Updates + Fix go vet warnings + pprof production security hardening#277

Closed
devfeel wants to merge 20 commits intomasterfrom
aicode
Closed

Go 1_21 Upgrade + Dependency Updates + Fix go vet warnings + pprof production security hardening#277
devfeel wants to merge 20 commits intomasterfrom
aicode

Conversation

@devfeel
Copy link
Owner

@devfeel devfeel commented Mar 2, 2026

v1.7.3 Release Notes

Highlights

Go 1_21 Upgrade + Dependency Updates + Fix go vet warnings + pprof production security hardening


⬆️ Upgrade

  • Go version upgraded to 1_21
  • golang_org/x/net upgraded to v0_23_0

🐛 Bug Fixes

Issue Description
#276 Resolve go vet warnings across codebase
- Fix loop variable capture in cache_runtime_test_go
- Fix sync_Pool copy warning in server_test_go
- Remove unreachable code in session/session_go

🔒 Security

Change Description
- pprof is now only enabled in development mode

⚠️ Breaking Change: pprof is no longer available in production by default. To enable, explicitly set RunMode = "development".

// Enable pprof (development only)
app_SetDevelopmentMode()
app_SetPProfConfig(true, 8080)

🔧 Improvements

  • Use keyed fields for HideReaddirFS struct in router_go
  • Improve code clarity and reduce warnings

2026.03.02 at ShangHai.

devfeel and others added 20 commits March 1, 2026 15:44
# Conflicts:
#	.github/workflows/go.yml
This reverts commit 6f56dfa.
chore: upgrade go to 1.21 and x/net to v0.23.0
- Fix loop variable capture in cache_runtime_test.go
- Use keyed fields for HideReaddirFS struct in router.go
- Remove unreachable code in session/session.go
- Fix sync.Pool copy warning in server_test.go
- Only enable pprof in development mode (security)
fix: resolve go vet warnings and add pprof production safety
@devfeel devfeel closed this Mar 2, 2026
@devfeel devfeel deleted the aicode branch March 2, 2026 08:00
@devfeel devfeel restored the aicode branch March 2, 2026 08:00
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.

1 participant