Skip to content

support shasum validation #6

Closed
Closed

Description

@springmeyer

Shasum checks are a first step towards:

  • Catching early partial/corrupt downloads, and;
  • Avoiding accepting binaries that have been tampered with.

TODO:

  • - research how npm handles unpacking tarballs to ensure we're using the most robust methods possible.

Activity

  1. springmeyer commented on Oct 17, 2014

    @springmeyer
    ContributorAuthor

    I hit spurious shasum checking errors consistently with npm so my vision is clouded by disliking this validation. I presume the npm problems are due to a lag between the availability of the published package and the updated shasum (is npm serving them from different caching endpoints?). I've not encountered a case yet or heard of reports where a node-pre-gyp download from https S3 was partial or corrupt. This certainly may be because we don't validate it (what this ticket was about) however I'm strongly inclined now to leave this feature off until someone steps up with a proposal that isboth:

    • solid security-wise
    • will not create a potential for spurious errors - e.g. only breaks when the network causes truncation of downloads or the binaries are actually tampered with.
    • has good tests
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

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions