Skip to content

crypto.publicDecrypt doesn't seem to work #30237

Closed
Closed
crypto.publicDecrypt doesn't seem to work#30237
Labels
cryptoIssues and PRs related to the crypto subsystem.questionIssues asking questions about Node.js.

Description

@yovanoc
yovanoc
opened on Nov 3, 2019
  • Version: 13.0.1
  • Platform: MacOS Catalina

image

pub doesn't work. Only priv and bob.publicKey but I know that priv should work because in docs: Because RSA public keys can be derived from private keys, a private key may be passed instead of a public key. I really want to know why pub isn't working.

It's showing that:

image

Activity

  1. tniessen
    added
    cryptoIssues and PRs related to the crypto subsystem.
    questionIssues asking questions about Node.js.
    on Nov 4, 2019

  2. bnoordhuis commented on Nov 4, 2019

    @bnoordhuis
    bnoordhuis
    on Nov 4, 2019
    Member

    @tniessen Is that an oversight? The 'private' here looks wrong to me:

    node/lib/internal/crypto/cipher.js

    Lines 68 to 69 in c80771d

    const publicDecrypt = rsaFunctionFor(_publicDecrypt, RSA_PKCS1_PADDING,
    'private');

    Since that means preparePrivateKey() is called, not preparePublicOrPrivateKey().

  3. tniessen
    added a commit that references this issue on Nov 4, 2019

    crypto: fix key requirements in asymmetric cipher

    a97850a

  4. tniessen commented on Nov 4, 2019

    @tniessen
    tniessen
    on Nov 4, 2019
    Member

    @bnoordhuis I think so, yes. I am surprised it went unnoticed for so long, but it is probably a rarely used function.

  5. tniessen
    closed this as completedin 78dbe74on Nov 13, 2019
  6. MylesBorins
    added a commit that references this issue on Nov 17, 2019

    crypto: fix key requirements in asymmetric cipher

    39d0a25
  7. targos
    added a commit that references this issue on Dec 1, 2019

    crypto: fix key requirements in asymmetric cipher

    be0b53d
  8. MylesBorins
    added a commit that references this issue on Dec 17, 2019

    crypto: fix key requirements in asymmetric cipher

    bb7e78a

  9. sda3211 commented on Jul 14, 2020

    @sda3211
    sda3211
    on Jul 14, 2020 · Hidden as spam

  10. sda3211 commented on Jul 14, 2020

    @sda3211
    sda3211
    on Jul 14, 2020 · Hidden as spam
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

    cryptoIssues and PRs related to the crypto subsystem.questionIssues asking questions about Node.js.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions