Skip to content

ETT-1533 plack-lib uses ancient Net::CIDR::Lite with multiple CVEs#218

Merged
moseshll merged 2 commits into
mainfrom
ETT-1533_net_cidr_lite
Jun 10, 2026
Merged

ETT-1533 plack-lib uses ancient Net::CIDR::Lite with multiple CVEs#218
moseshll merged 2 commits into
mainfrom
ETT-1533_net_cidr_lite

Conversation

@moseshll

@moseshll moseshll commented Jun 3, 2026

Copy link
Copy Markdown
Contributor
  • Remove offending module (after verifying that our vendored copy was unmodified)
  • Install Net::CIDR::Lite via cpanm -- the official Debian package (0.22-3) lags latest (0.24) which addresses additional CVEs
  • Remove outdated Plack::Builder::Conditionals (0.03) and install Debian package (0.05-2).
    • No CVEs here, only minor fixes.
  • Remove addition to plack-lib LICENSE covering these two modules.
  • Bump top-level LICENSE year (since it is routinely overlooked).

Note: nebula changes are in effect; this is safe to merge

- Remove offending module (after verifying that our vendored copy was unmodified)
- Install `Net::CIDR::Lite` via `cpanm` -- the official Debian package (0.22-3) lags latest (0.24)
  which addresses additional CVEs
- Remove outdated `Plack::Builder::Conditionals` (0.03) and install Debian package (0.05-2).
  - No CVEs here, only minor fixes.
- Remove addition to `plack-lib` LICENSE covering these two modules.
- Bump top-level LICENSE year (since it is routinely overlooked).
@moseshll moseshll marked this pull request as ready for review June 10, 2026 14:51
@moseshll moseshll requested a review from aelkiss June 10, 2026 14:52

@aelkiss aelkiss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Per @rrotter on the nebula PR we installed the Debian package for libnet-cidr-lite-perl rather than using the CPAN version there; the Debian version is up-to-date with security fixes. It would probably be good to keep consistent with nebula for what we do in the dockerfile. Other than that, this looks fine.

@moseshll moseshll merged commit 973ce33 into main Jun 10, 2026
3 checks passed
@moseshll moseshll deleted the ETT-1533_net_cidr_lite branch June 10, 2026 17:37
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