Request Context missing auth info

[LaurenSpiegel]

It appears that the requestContext being received by Vault does not contain the authType and signature version even though being set here -- https://github.com/scality/Arsenal/blob/master/lib/auth/auth.js#L67

Needs further investigation.

[ghost]

For the investigation, can you try it with the Auth API Refactor branches ? maybe it fixes things, or maybe it would be the opportunity to fix it ?

[alexandre-merle]

@DavidPineauScality yes probably solved in your pull request, because this line is wrong actually, res contain a field data, and that field contains signatureVersion and authType, so here we set values to undefined :)

[LaurenSpiegel]

I agree with Alex. This fixes it -- https://github.com/scality/Arsenal/pull/119/files#diff-4046a881f95c5cd889f1b21f47ac0d40R114

Plus you return all of the info (including signature age) in the data object. What is the status on merging this?

Along with it we can merge a test in integration which uses a test with a sig version condition.
@nicolas2bert

[ghost]

As far as I remember, it's pretty close to be Okay (at least for this first
step), but the ongoing GA effort is stalling that for now. Hopefully
mergeable next week ? I'll need your and alexandre's reviews, obviously.

On Wed, Sep 7, 2016 at 7:32 PM, Lauren Spiegel [email protected]
wrote:

I agree with Alex. This fixes it -- https://github.com/scality/
Arsenal/pull/119/files#diff-4046a881f95c5cd889f1b21f47ac0d40R114

Plus you return all of the info (including signature age) in the data
object. What is the status on merging this?

Along with it we can merge a test in integration which uses a test with a
sig version condition.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#154 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANpZZ1_iwVoBzNvu6wigFchN5aVdT5TKks5qnvUZgaJpZM4J2eur
.

David Pineau
Scality R&D Engineer

http://bit.ly/2bHLMkp

[ghost]

@LaurenSpiegel Oh my bad... There's one issue...
UTAPI is now using Arsenal's API, so this needs a branch/PR Too....

[alexandre-merle]

Fixed by #119