feathersjs-ecosystem / feathers-authentication-ldap

LDAP authentication strategy for feathers-authentication using Passport
MIT License
19 stars 4 forks source link

An in-range update of passport-ldapauth is breaking the build 🚨 #36

Closed greenkeeper[bot] closed 5 years ago

greenkeeper[bot] commented 5 years ago

The dependency passport-ldapauth was updated from 2.1.0 to 2.1.1.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

passport-ldapauth is a direct dependency of this project, and it is very likely causing it to break. If other packages depend on yours, this update is probably also breaking those in turn.

Status Details - ❌ **continuous-integration/travis-ci/push:** The Travis CI build could not complete due to an error ([Details](https://travis-ci.org/feathersjs-ecosystem/feathers-authentication-ldap/builds/462979295?utm_source=github_status&utm_medium=notification)).

Commits

The new version differs by 5 commits.

See the full diff

FAQ and help There is a collection of [frequently asked questions](https://greenkeeper.io/faq.html). If those don’t help, you can always [ask the humans behind Greenkeeper](https://github.com/greenkeeperio/greenkeeper/issues/new).

Your Greenkeeper Bot :palm_tree:

greenkeeper[bot] commented 5 years ago

After pinning to 2.1.0 your tests are still failing. The reported issue might not affect your project. These imprecisions are caused by inconsistent test results.

greenkeeper[bot] commented 5 years ago

Your tests are passing again with this update. Explicitly upgrade to this version πŸš€

Commits

The new version differs by 7 commits.

  • 856c5d0 2.1.2: Run error handler only once
  • 614fb36 Changefile
  • 0331fba Bump deps
  • 8ab7662 Merge pull request #80 from ddolcimascolo/error-handler-called-twice
  • a6a51e6 Remove Gemnasium badge since the gitlab integration requires too much access rights
  • ad6130f Update issue template
  • ae7d002 Only catch the 'error' event once.

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests are passing again with this update. Explicitly upgrade to this version πŸš€

Release Notes for Allow configuring HTTP status of missing credentials

Publish #86 which allows configuring the HTTP status code of missing credentials failure response

Commits

The new version differs by 13 commits.

  • ecb44fe 2.1.3: Allow configuring HTTP status of missing credentials
  • 2a635fa Add @ddolcimascolo to contributors
  • 0e3d514 Changelog for 2.1.3
  • 7ce0402 Update deps
  • 395ed09 Merge pull request #86 from ddolcimascolo/configureMissingCredentialsStatusCode
  • 88bb8e5 Support options.missingCredentialsStatus
  • 856c5d0 2.1.2: Run error handler only once
  • 614fb36 Changefile
  • 0331fba Bump deps
  • 8ab7662 Merge pull request #80 from ddolcimascolo/error-handler-called-twice
  • a6a51e6 Remove Gemnasium badge since the gitlab integration requires too much access rights
  • ad6130f Update issue template
  • ae7d002 Only catch the 'error' event once.

See the full diff

greenkeeper[bot] commented 4 years ago

Your tests are passing again with this update. Explicitly upgrade to this version πŸš€

Commits

The new version differs by 16 commits.

  • 0330982 2.1.4: Allow any version of @types/node
  • 238539f Allow any version of node types
  • 0467cc3 Update merge (dependency of watch) to 1.2.1 to get rid of security alert
  • ecb44fe 2.1.3: Allow configuring HTTP status of missing credentials
  • 2a635fa Add @ddolcimascolo to contributors
  • 0e3d514 Changelog for 2.1.3
  • 7ce0402 Update deps
  • 395ed09 Merge pull request #86 from ddolcimascolo/configureMissingCredentialsStatusCode
  • 88bb8e5 Support options.missingCredentialsStatus
  • 856c5d0 2.1.2: Run error handler only once
  • 614fb36 Changefile
  • 0331fba Bump deps
  • 8ab7662 Merge pull request #80 from ddolcimascolo/error-handler-called-twice
  • a6a51e6 Remove Gemnasium badge since the gitlab integration requires too much access rights
  • ad6130f Update issue template

There are 16 commits in total.

See the full diff