Skip to content

Pin production nan to an exact version #6

Description

@enochgroot

package.json ships "nan": "^2.23.1". NAN is compiled into the addon (binding.gyp require('nan')). A caret is not a pin: every consumer npm install re-resolves >=2.23.1 <3.0.0. This repo's lockfile hashing 2.28.0 does not bind dependents.

Pin nan to the reviewed exact version (currently 2.28.0). Prefer a shrinkwrap (or equivalent dependents actually apply) with integrity hashes. CI should install from the lockfile (npm ci), not a floating npm install.

Activity

  1. enochgroot commented on Sep 21, 2026

    @enochgroot
    OwnerAuthor

    Implemented in #14 (branch fix/review-open-6-13).

  2. enochgroot commented on Sep 22, 2026

    @enochgroot
    OwnerAuthor

    Tracked upstream: msgpack#25696

    Closing this fork issue as a duplicate of that ticket. Numbering on enochgroot/msgpack-node is a different sequence from msgpack/msgpack-node.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions