Skip to content

Update bcrypt to the latest version 🚀 #4703

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 10, 2018
Merged

Conversation

greenkeeper[bot]
Copy link
Contributor

@greenkeeper greenkeeper bot commented Apr 7, 2018

☝️ Greenkeeper’s updated Terms of Service will come into effect on April 6th, 2018.

Version 2.0.0 of bcrypt was just published.

Dependency bcrypt
Current Version 1.0.3
Type optionalDependency

The version 2.0.0 is not covered by your current version range.

If you don’t accept this pull request, your project will work just like it did before. However, you might be missing out on a bunch of new features, fixes and/or performance improvements from the dependency update.

It might be worth looking into these changes and trying to get this project onto the latest version of bcrypt.

If you have a solid test suite and good coverage, a passing build is a strong indicator that you can take advantage of these changes directly by merging the proposed change into your project. If the build fails or you don’t have such unconditional trust in your tests, this branch is a great starting point for you to work on the update.


Commits

The new version differs by 26 commits.

  • ab026b2 v2.0.0
  • f00d4b8 Merge pull request #589 from agathver/libc-aware
  • dab435e install and use any-promise (#504)
  • 9a9ab45 Make binaries libc aware
  • aac593c Merge pull request #587 from agathver/hash-version-support
  • 2d45be1 Allow to choose bcrypt minor version
  • 0ea1b36 Merge pull request #549 from agathver/2b-hashes
  • 4c44f20 Add support for $2b$ hashes
  • e8cde51 Merge pull request #583 from ofrobots/async-resource
  • 6a79eaf fix: propagate async context
  • 88590ea Merge pull request #584 from kelektiv/snyk-fix-bc668290
  • 1da0f44 fix: package.json to reduce vulnerabilities
  • 43734e3 Merge pull request #564 from david-a-wheeler/readme-timing
  • f2bec20 README: comparisons resist timing attacks
  • 096a34f Merge pull request #554 from agathver/node-9

There are 26 commits in total.

See the full diff

FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.


Your Greenkeeper bot 🌴

@codecov
Copy link

codecov bot commented Apr 7, 2018

Codecov Report

Merging #4703 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4703      +/-   ##
==========================================
- Coverage   92.71%   92.65%   -0.06%     
==========================================
  Files         119      119              
  Lines        8581     8581              
==========================================
- Hits         7956     7951       -5     
- Misses        625      630       +5
Impacted Files Coverage Δ
src/Adapters/Cache/InMemoryCache.js 91.66% <0%> (-8.34%) ⬇️
src/Adapters/Auth/meetup.js 84.21% <0%> (-5.27%) ⬇️
src/RestWrite.js 93.41% <0%> (-0.37%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0fd8493...3c52977. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant