Compare commits

..

No commits in common. "4a20fac0afaa5b3de93ae692ef118b628eabe970" and "b2d56dfeff321f73d0ff4afad4f6169ab9b5ea11" have entirely different histories.

2 changed files with 2 additions and 29 deletions

View File

@ -32,15 +32,6 @@ function setAuthHandler(handler) {
return auth;
}
function decodeHeader(value) {
// extract string after 'x-amz-meta-scality-rewritten-'
let tail = value.slice(29);
tail = tail.replace(/\|/g, '/');
tail = tail.replace(/-/g, '=');
tail = Buffer.from(tail, 'base64').toString();
return `x-amz-meta-${tail}`;
}
/**
* This function will check validity of request parameters to authenticate
*
@ -60,22 +51,7 @@ function decodeHeader(value) {
*/
function extractParams(request, log, awsService, data) {
log.trace('entered', { method: 'Arsenal.auth.server.extractParams' });
const modifiedRequest = Object.assign({}, request);
modifiedRequest.headers = {};
Object.keys(request.headers).forEach(headerName => {
modifiedRequest.headers[headerName] = request.headers[headerName];
});
for (let i = 0; i < modifiedRequest.rawHeaders.length; i += 2) {
const headerName = modifiedRequest.rawHeaders[i];
const lowerCaseHeaderName = headerName.toLowerCase();
if (lowerCaseHeaderName.startsWith('x-amz-meta-scality-rewritten-')) {
const decodedHeader = decodeHeader(headerName);
modifiedRequest.headers[decodedHeader.toLowerCase()] =
request.headers[lowerCaseHeaderName];
delete modifiedRequest.headers[lowerCaseHeaderName];
}
}
const authHeader = modifiedRequest.headers.authorization;
const authHeader = request.headers.authorization;
let version = null;
let method = null;
@ -109,8 +85,7 @@ function extractParams(request, log, awsService, data) {
return { err: errors.NotImplemented };
}
log.trace('identified auth method', { version, authMethod: method });
return checkFunctions[version][method](modifiedRequest, log,
data, awsService);
return checkFunctions[version][method](request, log, data, awsService);
}
// no auth info identified

View File

@ -19,7 +19,6 @@ describe('Public Access', () => {
const request = {
method: 'GET',
headers: { host: 's3.amazonaws.com' },
rawHeaders: [],
url: '/bucket',
query: {},
};
@ -47,7 +46,6 @@ describe('Public Access', () => {
host: 's3.amazonaws.com',
authorization: 'noAuth',
},
rawHeaders: ['authorization', 'noAuth'],
url: '/bucket',
query: {},
};