Skip to content

BUILDING.md references p. 60 of a 40 page security policy document #17536

Closed
Closed
BUILDING.md references p. 60 of a 40 page security policy document#17536
Labels
buildIssues and PRs related to Node.js builds or CI infrastructure.docIssues and PRs related to Node.js documentation.opensslIssues and PRs related to the OpenSSL dependency.

Description

@rob3c
rob3c
opened on Dec 7, 2017
  • Version: N/A
  • Platform: N/A
  • Subsystem: N/A

The current BUILDING.md instruction page in master on building node with FIPS compliant OpenSSL says:
See the security policy page 60 for more details.
Unfortunately, the current security policy pdf at the link only has 40 pages, so it's unclear what is being referenced.

Activity

  1. addaleax
    added
    buildIssues and PRs related to Node.js builds or CI infrastructure.
    docIssues and PRs related to Node.js documentation.
    opensslIssues and PRs related to the OpenSSL dependency.
    on Dec 7, 2017

  2. addaleax commented on Dec 7, 2017

    @addaleax
    addaleax
    on Dec 7, 2017
    Member

    /cc @nodejs/crypto

  3. Trott commented on Dec 7, 2017

    @Trott
    Trott
    on Dec 7, 2017
    Member

    The phrase "validated module" does not appear in the PDF. I'll open a PR to remove the entire sentence, as the policy is linked to later in the paragraph anyway.

  4. bnoordhuis commented on Dec 7, 2017

    @bnoordhuis
    bnoordhuis
    on Dec 7, 2017
    Member

    Pinging @mhdawson, looks like you added that in #2278.

  5. Trott
    added a commit that references this issue on Dec 7, 2017

    doc: simplify and clarify FIPS text in BUILDING.md

    34249b2

  6. Trott commented on Dec 7, 2017

    @Trott
    Trott
    on Dec 7, 2017
    Member

    PR to update text: #17538

  7. Trott
    added a commit that references this issue on Dec 10, 2017

    doc: simplify and clarify FIPS text in BUILDING.md

    4426d75
  8. Trott
    closed this as completedin 4426d75on Dec 10, 2017
  9. MylesBorins
    added 2 commits that reference this issue on Dec 12, 2017

    doc: simplify and clarify FIPS text in BUILDING.md

    30f5a26

    doc: simplify and clarify FIPS text in BUILDING.md

    a965dda
  10. gibfahn
    added 4 commits that reference this issue on Dec 20, 2017

    doc: simplify and clarify FIPS text in BUILDING.md

    dcee5ed

    doc: simplify and clarify FIPS text in BUILDING.md

    8e4cb1b

    doc: simplify and clarify FIPS text in BUILDING.md

    0b45766

    doc: simplify and clarify FIPS text in BUILDING.md

    26e0fa8
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

    buildIssues and PRs related to Node.js builds or CI infrastructure.docIssues and PRs related to Node.js documentation.opensslIssues and PRs related to the OpenSSL dependency.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions