Skip to content

Build seidb in the sidecar image with Go 1.27.3 - #618

Merged
masih merged 2 commits into
mainfrom
masih/1792524934-go-1.27.3
Oct 20, 2026
Merged

masih merged 2 commits into
mainfrom
masih/1792524934-go-1.27.3

Conversation

@masih

@masih masih commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Moves the seidb stage of sidecar/Dockerfile from golang:1.27.1 to golang:1.27.3, which picks up the Go 1.27.2 security and compiler fixes and the 1.27.3 crypto/x509 security fix. The stage still builds sei-chain at the same SEIDB_REF. That ref's go.mod says go 1.27.1, and seidb builds from it with Go 1.27.3. The other stages already use the floating golang:1.27 tag. This matches the sei-chain bump in sei-protocol/sei-chain#4576.

@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access that start with 'Devin'.

  • Disable automatic comment, CI, and merge conflict monitoring

@masih
masih marked this pull request as ready for review October 20, 2026 22:01
@cursor

cursor Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
Single Dockerfile base-image pin for the seidb compile stage; no application or runtime logic changes.

Overview
Bumps the seidb Docker build stage in sidecar/Dockerfile from golang:1.27.1 to golang:1.27.3, so the bundled seidb binary is compiled with Go 1.27.2/1.27.3 security and compiler fixes (including the crypto/x509 fix in 1.27.3).

SEIDB_REF and the rest of the sidecar image build are unchanged; the main build stage still uses the floating golang:1.27 tag. This aligns with the sei-chain Go bump in sei-chain#4576.

Reviewed by Cursor Bugbot for commit c538a57. Bugbot is set up for automated code reviews on this repo. Configure here.

@seidroid seidroid Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This bumps the seidb build stage in sidecar/Dockerfile from golang:1.27.1 to golang:1.27.3, keeping the same SEIDB_REF. The golang:1.27.3 tag exists and is newer than the pinned ref's go 1.27.1 directive, so the toolchain resolves without a download and nothing blocks; the one independent reading (codex) found nothing, which matches this review.

seidroid review · decision approve · session 80880581207a4c38bfdc7dbbc598c690 · turn resp_claude_ba826ecea42b0d59bbf821bb94bfc449 · item 0c516d3f9f6a5afb8c766f34f1add993

Findings: 0 blocking | 0 non-blocking | 0 posted inline

@masih
masih merged commit 1168afa into main Oct 20, 2026
18 of 19 checks passed
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