Skip to content

Commit

Permalink
Bug fix - passport should not attempt to initialize if no auth provid…
Browse files Browse the repository at this point in the history
…er is present
  • Loading branch information
arobson committed Dec 18, 2014
1 parent a08a05d commit 70fb331
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
## 0.3.0
Primary motivation here is to begin work on a version of autohost that will work well with a hypermedia library ( [hyped](https://github.com/leankit-labs/hyped) ). This is a breaking change because of several structural and naming changes to how resources get modeled.

### prerelease 15
Bug fix - passport should not attempt to initialize when no auth provider is passed to init.

### prerelease 14
* Bug fix - auth middleware shouldn't get added to sockets twice
* Bug fix - user middleware should always come after auth middleware
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "autohost",
"version": "0.3.0-14",
"version": "0.3.0-15",
"description": "Resource driven, transport agnostic host",
"main": "src/index.js",
"dependencies": {
Expand Down
4 changes: 3 additions & 1 deletion src/http/adapter.js
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,9 @@ function wireupAction( resource, actionName, action, meta, resources ) {
module.exports = function( cfg, auth, httpLib, req, meter ) {
config = cfg;
authStrategy = auth;
passport = passportFn( cfg, auth, meter );
if( auth ) {
passport = passportFn( cfg, auth, meter );
}
http = httpLib;
metrics = meter;
HttpEnvelope = require( './httpEnvelope.js' )( req );
Expand Down

0 comments on commit 70fb331

Please sign in to comment.