Skip to content

[security] Mitigate eMudhra/emSign CA incident in dependency resolution, CI, and CLI installs : Issue #79 - #81

Open
Shreyavemana wants to merge 32 commits into
nitrocloudofficial:developfrom
Shreyavemana:security/emsign-mitigation
Open

Shreyavemana wants to merge 32 commits into
nitrocloudofficial:developfrom
Shreyavemana:security/emsign-mitigation

Conversation

@Shreyavemana

Copy link
Copy Markdown
Collaborator

Summary

Mitigates the eMudhra/emSign CA certificate security incident by updating certificate dependencies and securing pip/uv installation paths across CI, publishing workflows, and the CLI.

Changes

  • Updated certifi to 2026.10.19 or later in dependency declarations and uv.lock.
  • Added SHA-256-pinned pip 26.2.2 bootstrapping to test and publish workflows.
  • Updated nitrostack-py install to securely upgrade pip before installing packages and reject uv versions older than 0.13.4.
  • Added security guidance to the README.

Tests

  • Added tests/test_cli_install_security.py for pip bootstrapping, version parsing, uv version checks, and failure handling.
  • Added tests/test_security_config.py for dependency versions and CI workflow configuration.
  • Updated tests/test_cli_install.py to accommodate the new version checks.
  • 42 targeted tests passed.
  • Full test suite: 859 passed.
  • Python 3.10 verification and Ruff checks were attempted but did not complete because of test-environment/tooling issues (pytest-timeout was initially unavailable; Ruff was not installed).

Documentation

Added a README section covering the incident, minimum fixed versions, and recommendations to update existing virtual environments, refresh lockfiles, and rebuild affected container images.

manish-wekan and others added 30 commits September 10, 2026 18:05
…ase PR.

Keep pyproject.toml and uv.lock in sync, skip a bump while a version is still unpublished, and only hit the PyPI environment when an upload will actually run.
Bring the bump/publish workflows in line with develop (via main) so the PR can merge.
…ure/release-automation

Automate version bumps and PyPI publishes from main
.env values were written over os.environ unconditionally, so the PORT=3000
that `nitrostack init` puts in .env overrode the PORT a host passes when
starting the server. NitroStudio starts a second project on 3001; it bound
3000 anyway and died with "address already in use" (NITSTD-222).

Keys already in the process environment are now kept, as with
python-dotenv and Node's dotenv. .env still fills keys the process
doesn't set, and defaults stay lowest.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…process-env-wins-over-dotenv

ConfigService: process environment wins over .env
publish.yml only uploads through PyPI trusted publishing, which has never
run here, and the Bump version workflow can't open its release PR (the org
doesn't allow GitHub Actions to create pull requests). The script covers
both: optionally makes release/<next> from origin/main (--bump), checks the
branch matches origin, the lockfile matches and the version is new, runs the
tests, builds, twine-checks, uploads after asking, verifies a clean install
from PyPI, and opens the release PR. `--tag` tags origin/main after the merge.
`--dry-run` stops before anything leaves the machine.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Run from a temp copy: --bump switches branches, which can rewrite the
  script while bash is still reading it.
- --bump plans from origin/main's files in a scratch folder and checks
  everything before switching, so a failed check no longer leaves you on a
  detached origin/main.
- Only a 404 counts as "not on PyPI"; offline or PyPI errors stop the run
  instead of passing the "version is new" check.
- No terminal and no --yes means "no" instead of a silent exit.
- Tag the uploaded commit right after the upload (no separate step that
  could tag later, unreleased merges on main). --tag stays for catching up
  untagged releases and prefers the release branch head.
- A failed PR creation after the upload prints the manual step.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The README is the PyPI project page. Rewrite it for 0.4.x: building
blocks (tools, resources, prompts, guards, auth, caching, rate limits,
tasks, events, health), env precedence, templates, clients.

tests/test_readme_examples.py compiles every python block, checks every
nitrostack import exists, and runs the Quick Start and Building Blocks
examples through the testing harness so the page can't drift.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e/release-script

Add scripts/release.sh for releasing to PyPI by hand
…/readme-refresh

docs: refresh README (PyPI page) for the current SDK
`nitrostack-py upgrade` failed on every project `init` creates: the
dependency pattern also matched `nitrostack = { path = "vendor/nitrostack" }`
in [tool.uv.sources] and produced an invalid pyproject.toml. Spec rewrites
are now limited to [project].dependencies, and the pattern skips paths
ending in /nitrostack.

Fixing that alone isn't enough: a vendored project runs vendor/nitrostack
whatever spec it declares, and uv accepts a spec the vendored copy doesn't
satisfy. For vendored projects, upgrade now replaces vendor/nitrostack
with the target release (copied from the running CLI when it is that
release, otherwise the PyPI wheel, checked against its sha256). The new
tree is built beside the old one and swapped in, so a failed download
leaves the project as it was.

init's next steps said NitroStudio can't open Python projects; it can.
The pizzaz open-shops hint is printed only for the pizzaz template.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…upgrade-vendored

Fix upgrade on vendored projects; correct init's NitroStudio note
`init` vendors the SDK into vendor/nitrostack. `validate` walked it as
project code, and nitrostack/testing applies @mcp_app to a class defined
inside a function, so every vendored project that imported
nitrostack.testing failed with "uses @mcp_app but no decorated
application class was found after import". Compose's typecheck runs
`validate`, so a NitroStudio build could not get past it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…validate-skip-vendored-sdk

validate: skip the vendored SDK
Align Python pack with the TypeScript CLI: optional inclusion of .env
files in the wheel, with a warning that lists paths without printing values.
…/pack-include-env

pack: add --include-env
…/producthunt-badge

docs: add Product Hunt featured badge to README
…/producthunt-top-post-badge

docs: update Product Hunt badge to nitrocloudofficial#1 Product of the Day top-post badge
@Shreyavemana
Shreyavemana changed the base branch from main to develop October 23, 2026 14:37

This branch has not been deployed

No deployments
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.

4 participants