Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

crypto: pbkdf2 should keep current domain #5819

Closed
wants to merge 1 commit into from
Closed

crypto: pbkdf2 should keep current domain #5819

wants to merge 1 commit into from

Conversation

rustyconover
Copy link

Passes the active domain into the callback for crypto.pbkdf2 function, so that any
exceptions are thrown to the domain handler.

Fixes issue #5801. Please review.

Fixes issue #5801.  Passes the active domain into
the callback for crypto.pbkdf2 function, so that any
exceptions are thrown to the domain handler.
@Nodejs-Jenkins
Copy link

Thank you for contributing this pull request! Here are a few pointers to make sure your submission will be considered for inclusion.

The following commiters were not found in the CLA:

  • Rusty Conover

You can fix all these things without opening another issue.

Please see CONTRIBUTING.md for more information

@tjfontaine
Copy link

fixed in a slightly different way in 6e689ec

@tjfontaine tjfontaine closed this Sep 16, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants