Extend AccessToken to parse Basic auth headers
Allow convenient URLs for curl and browsers such as: - http://some-long-token@localhost:3000/ - http://token:some-long-token@localhost:3000/ Basic Auth specifies a 'Basic' scheme for the Authorization header similar to how OAuth specifies 'Bearer' as an auth scheme. Following a similar convention, extract the access token from the Authorization header when it specifies the 'Basic' scheme, assuming it is the larger of the <user>:<pass> segments.
This commit is contained in:
parent
83d8844b70
commit
fbb091e3b3
|
@ -189,6 +189,20 @@ module.exports = function(AccessToken) {
|
||||||
// Decode from base64
|
// Decode from base64
|
||||||
var buf = new Buffer(id, 'base64');
|
var buf = new Buffer(id, 'base64');
|
||||||
id = buf.toString('utf8');
|
id = buf.toString('utf8');
|
||||||
|
} else if (/^Basic /i.test(id)) {
|
||||||
|
id = id.substring(6);
|
||||||
|
id = (new Buffer(id, 'base64')).toString('utf8');
|
||||||
|
// The spec says the string is user:pass, so if we see both parts
|
||||||
|
// we will assume the longer of the two is the token, so we will
|
||||||
|
// extract "a2b2c3" from:
|
||||||
|
// "a2b2c3"
|
||||||
|
// "a2b2c3:" (curl http://a2b2c3@localhost:3000/)
|
||||||
|
// "token:a2b2c3" (curl http://token:a2b2c3@localhost:3000/)
|
||||||
|
// ":a2b2c3"
|
||||||
|
var parts = /^([^:]*):(.*)$/.exec(id);
|
||||||
|
if (parts) {
|
||||||
|
id = parts[2].length > parts[1].length ? parts[2] : parts[1];
|
||||||
|
}
|
||||||
}
|
}
|
||||||
return id;
|
return id;
|
||||||
}
|
}
|
||||||
|
|
|
@ -39,6 +39,48 @@ describe('loopback.token(options)', function() {
|
||||||
.end(done);
|
.end(done);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
describe('populating req.toen from HTTP Basic Auth formatted authorization header', function() {
|
||||||
|
it('parses "standalone-token"', function(done) {
|
||||||
|
var token = this.token.id;
|
||||||
|
token = 'Basic ' + new Buffer(token).toString('base64');
|
||||||
|
createTestAppAndRequest(this.token, done)
|
||||||
|
.get('/')
|
||||||
|
.set('authorization', this.token.id)
|
||||||
|
.expect(200)
|
||||||
|
.end(done);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('parses "token-and-empty-password:"', function(done) {
|
||||||
|
var token = this.token.id + ':';
|
||||||
|
token = 'Basic ' + new Buffer(token).toString('base64');
|
||||||
|
createTestAppAndRequest(this.token, done)
|
||||||
|
.get('/')
|
||||||
|
.set('authorization', this.token.id)
|
||||||
|
.expect(200)
|
||||||
|
.end(done);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('parses "ignored-user:token-is-password"', function(done) {
|
||||||
|
var token = 'username:' + this.token.id;
|
||||||
|
token = 'Basic ' + new Buffer(token).toString('base64');
|
||||||
|
createTestAppAndRequest(this.token, done)
|
||||||
|
.get('/')
|
||||||
|
.set('authorization', this.token.id)
|
||||||
|
.expect(200)
|
||||||
|
.end(done);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('parses "token-is-username:ignored-password"', function(done) {
|
||||||
|
var token = this.token.id + ':password';
|
||||||
|
token = 'Basic ' + new Buffer(token).toString('base64');
|
||||||
|
createTestAppAndRequest(this.token, done)
|
||||||
|
.get('/')
|
||||||
|
.set('authorization', this.token.id)
|
||||||
|
.expect(200)
|
||||||
|
.end(done);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
it('should populate req.token from a secure cookie', function(done) {
|
it('should populate req.token from a secure cookie', function(done) {
|
||||||
var app = createTestApp(this.token, done);
|
var app = createTestApp(this.token, done);
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue