Skip to content

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

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

Shreyavemana wants to merge 2 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.

@Shreyavemana
Shreyavemana changed the base branch from main to develop October 23, 2026 14:37
@Shreyavemana
Shreyavemana force-pushed the security/emsign-mitigation branch from 294f548 to 8464d8a Compare October 24, 2026 05:21

@AadarshM07 AadarshM07 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.

The work seems to address issue #79 properly. All the tests also pass, and I think it is good to merge.

cc: @manish-wekan

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.

2 participants