chore: DEVPLAT-6523 bump deps to latest minor/patch within major#164
Merged
Conversation
Ran `go get -u ./...` and `go mod tidy` to pull in available minor/patch updates across direct and indirect dependencies. Notable direct bumps: dd-trace-go v2.4.0 → v2.8.1, grpc v1.77.0 → v1.81.0, redis/go-redis/v9 v9.17.2 → v9.19.0, franz-go v1.20.5 → v1.21.1, aws-sdk-go-v2 service modules, mage v1.15.0 → v1.17.2, plus minor logrus/protobuf/mysql bumps. getsentry/sentry-go is intentionally pinned at v0.40.0; v0.46.0 removed Event.Extra (used in pkg/tracking/sentry.go) and needs a separate migration to Event.Contexts. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
AndrewHart102
changed the title
v0.46.0 removed Event.Extra (getsentry/sentry-go#1274). Migrate the logrus hook to populate Event.Contexts under a "logrus" key, matching the official migration path for arbitrary structured data on error events. entry.Data fields now appear under a Logrus context section in the Sentry UI instead of the legacy Additional Data section. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 3d2b87c. Configure here.
AndrewHart102
changed the title
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- dd-trace-go v2 + contribs: v2.8.1 -> v2.8.2 - aws-sdk-go-v2 core/config/credentials/sts: latest patch - aws-sdk-go-v2 s3, sagemakerruntime, sfn, sqs: latest minor - smithy-go: v1.25.1 -> v1.27.0 - redis/go-redis/v9: v9.19.0 -> v9.20.0 - twmb/franz-go: v1.21.1 -> v1.21.2 - google.golang.org/grpc: v1.81.0 -> v1.81.1 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
PR to update patch and minor go dependencies.
DEVPLAT-6523
Testing considerations
This branch is currently deployed to Go-Chassis development. You can run these two steps to verify it's working.
Run this link to make sure HTTP requests working https://www.development.corp.scribd.com/chassis/go/example/1
Run the GraphQL queries to make sure GraphQL still works.
https://api.development.scribd.com/api/v3/playground
By sending this URL you will cause a failure.
https://www.development.corp.scribd.com/chassis/go/example?id=sadsad
It should show up here.
https://scribd.sentry.io/issues/7521435723/?environment=development&project=1800793&query=is%3Aunresolved&referrer=issue-stream&statsPeriod=24h
Checklist
developmentand/orstagingREADME.mdas necessaryRelated links
Note
Medium Risk
Wide dependency upgrades (tracing, AWS, gRPC, Sentry) can affect runtime behavior; the Sentry context change alters how log fields appear in issues but is localized to error reporting.
Overview
Bumps Go 1.26.2 → 1.26.4 in
Dockerfileandgo.mod, and refreshesgo.sumwith patch/minor updates across the stack (notably dd-trace-go 2.4 → 2.8, AWS SDK v2, gRPC, Sentry, Redis, franz-go, and related indirects).The only application change is in
pkg/tracking/sentry.go: logrus fields attached to Sentry events move fromExtrato aContexts["logrus"]block, matching newer sentry-go event shape so error metadata still appears in Sentry after the dependency bump.Reviewed by Cursor Bugbot for commit 0e67a0c. Bugbot is set up for automated code reviews on this repo. Configure here.