Skip to content

Commit c1c1e8b

Browse files
jBouyoudptitgraig
authored andcommitted
Make authentication optional (#5)
* optionnal auth * optional authentication
1 parent 74093f9 commit c1c1e8b

File tree

4 files changed

+17
-19
lines changed

4 files changed

+17
-19
lines changed

lib/download.js

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,18 +39,20 @@ function download(requestUrl, downloadPath, config) {
3939
randomize: true
4040
}, config.retry || {});
4141

42-
var auth = "Basic " + utils.getAuthToken(config);
42+
var auth = utils.getAuthToken(config);
4343

4444
var _request = request.defaults({
4545
proxy: parsedUrl.protocol === 'https:' ? config.httpsProxy : config.proxy,
4646
strictSSL: config.strictSsl,
47-
timeout: config.timeout,
48-
headers : {
49-
"Authorization" : auth
50-
}
47+
timeout: config.timeout
5148
});
49+
if(auth !== null){
50+
_request = _request.defaults({headers : {
51+
"Authorization" : auth
52+
}});
53+
}
5254

53-
_request = _request.defaults(config.request || {})
55+
_request = _request.defaults(config.request || {});
5456

5557
// Retry on network errors
5658
operation = retry.operation(retryOptions);

lib/extract.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,6 @@ function getExtractor(archive) {
111111
var type = mout.array.find(extractorTypes, function (type) {
112112
return mout.string.endsWith(archive, type);
113113
});
114-
console.log(type);
115114
return type ? extractors[type] : null;
116115
}
117116

lib/request.js

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,24 +10,25 @@ function requestWrapper(requestUrl, config) {
1010

1111
var protocol = url.parse(requestUrl).protocol;
1212

13-
var auth = "Basic " + utils.getAuthToken(config);
13+
var auth = utils.getAuthToken(config);
1414

1515
var _request = request.defaults({
1616
proxy: protocol === 'https:' ? config.httpsProxy : config.proxy,
1717
strictSSL: false,
18-
timeout: config.timeout,
19-
headers : {
18+
timeout: config.timeout
19+
});
20+
if(auth !== null){
21+
_request = _request.defaults({headers : {
2022
"Authorization" : auth
21-
}
22-
})
23+
}});
24+
}
2325

2426
_request = _request.defaults(config.request || {})
2527

2628
_request(requestUrl, function (error, response, body) {
2729
if (error) {
2830
deferred.reject(createError('Request to ' + requestUrl + ' failed: ' + error.message, error.code));
2931
} else {
30-
console.log('laencore');
3132
if (response.statusCode === 200) {
3233
deferred.resolve(body);
3334
} else {

lib/utils.js

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,9 @@ function extractReleases(response) {
3838

3939
function getAuthToken(config) {
4040
if (!config.auth) {
41-
throw createError('Authentication token not configured', 'ENOAUTHTOKEN', {
42-
details: 'You need to set the authentication token with `auth` field in .bowerrc'
43-
});
44-
} else {
45-
return config.auth;
41+
return null;
4642
}
47-
return
43+
return "Basic " + config.auth;
4844
}
4945

5046
function getRegistryUrl(config) {

0 commit comments

Comments
 (0)