Skip to content

fix(deps): build with Go 1.26.10 for the eMudhra CA blocklist (FB-5055) - #88

Merged
fstr merged 1 commit into
mainfrom
florian/fb-5055-go-1.26.10
Oct 24, 2026
Merged

fstr merged 1 commit into
mainfrom
florian/fb-5055-go-1.26.10

Conversation

@fstr

@fstr fstr commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

The published ghcr.io/firebolt-db/mcp-server:latest image (built 2026-01-26) trusts the compromised emSign ECC Root CA - G3 and AAA Certificate Services roots: its distroless CA bundle predates the Debian fix. The server makes TLS calls to the Firebolt API from customer machines, so a MITM with one of the leaked eMudhra intermediates would be accepted.

  • Bumps go in go.mod from 1.26.0 to 1.26.10, the first 1.26 patch with the eMudhra blocklist (CVE-2026-111943). setup-go installs exactly this version via go-version-file, so release binaries get it.
  • The Dockerfile stays on the floating gcr.io/distroless/base:nonroot. Its current digest (f9518b74…) is clean (checked by certificate subject with openssl), so the next release build picks up the fix.

fix: so release-please cuts a release. The image only gets rebuilt once the release PR (currently #80) is merged.

go build ./... and go test ./... pass locally.

🤖 Generated with Claude Code


Note

Medium Risk
Toolchain bump changes default TLS root trust for outbound API calls; scope is limited to the Go version pin with no application logic changes.

Overview
Bumps the required Go toolchain from 1.26.0 to 1.26.10 so CI and release builds (via go-version-file: go.mod in test/release workflows) compile with the first 1.26 patch that includes the eMudhra root CA blocklist (CVE-2026-111943).

This addresses trust in compromised eMudhra-related roots for outbound TLS to the Firebolt API; it does not change application code—only the minimum Go version declared in go.mod.

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

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@fstr
fstr merged commit 784fca2 into main Oct 24, 2026
11 checks passed
@fstr
fstr deleted the florian/fb-5055-go-1.26.10 branch October 24, 2026 21:53
kop pushed a commit that referenced this pull request Oct 26, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.7.0](v0.6.0...v0.7.0)
(2026-10-26)


### ⚠ BREAKING CHANGES

* **server:** add streamable-http transport
([#81](#81))

### Features

* **server:** add streamable-http transport
([#81](#81))
([9713ba4](9713ba4))


### Bug Fixes

* **deps:** build with Go 1.26.10 for the eMudhra CA blocklist (FB-5055)
([#88](#88))
([784fca2](784fca2))
* **server:** block cross-site tool calls and drain requests on shutdown
([#82](#82))
([bf68432](bf68432))
* validate database connection parameters
([#79](#79))
([d240c04](d240c04))


### Dependencies

* **github-actions:** bump the github-actions-deps group across 1
directory with 7 updates
([#77](#77))
([d9f59ba](d9f59ba))
* **gomod:** bump github.com/antchfx/xpath from 1.3.5 to 1.3.6
([#76](#76))
([08332f3](08332f3))
* **gomod:** bump github.com/klauspost/compress from 1.18.2 to 1.18.7
([#87](#87))
([feec202](feec202))
* **gomod:** bump golang.org/x/sync from 0.20.0 to 0.24.0 in the go-deps
group ([#85](#85))
([2bae7dd](2bae7dd))
* **gomod:** bump the go-deps group across 1 directory with 8 updates
([#84](#84))
([aafedab](aafedab))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Diff is version manifest, changelog, and README tag updates only; no
runtime code changes in this PR.
> 
> **Overview**
> **Release Please** bumps the project from **0.6.0** to **0.7.0** in
`.github/release-please/manifest.json`, adds the **0.7.0** section to
`CHANGELOG.md`, and updates pinned Docker image tags and release URLs in
`README.md` (install, Claude Desktop, Firebolt Core examples).
> 
> This PR does not change server code; it packages already-merged work.
The new changelog entry calls out a **breaking** **streamable-http**
transport, plus security/behavior fixes (cross-site tool-call blocking,
shutdown draining, DB connection validation), Go **1.26.10** for the
eMudhra CA blocklist, and routine dependency bumps.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
e33a069. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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