Skip to content
This repository has been archived by the owner on Apr 20, 2019. It is now read-only.

Timestamp Skew response #7

Closed
StarpTech opened this issue Jun 16, 2015 · 3 comments
Closed

Timestamp Skew response #7

StarpTech opened this issue Jun 16, 2015 · 3 comments
Assignees
Labels
Milestone

Comments

@StarpTech
Copy link
Contributor

Does scarecrow supports the timestamp skew response? When I use an invalid timestamp I only get a response like that without tm and tsm values.

Response:
{"statusCode":401,"error":"Unauthorized","message":"Error: Stale timestamp"}

ResponseHeaders:
content-type: application/json; charset=utf-8 cache-control: no-cache

The Boom object is correct but hapi 8.5.3 send no headers.

@StarpTech
Copy link
Contributor Author

I found the issue on line https://github.com/hueniverse/scarecrow/blob/master/lib/index.js#L86 you don't pass the boom object from hawk.

@hueniverse hueniverse added the bug label Jun 16, 2015
@hueniverse
Copy link
Contributor

Should just pass along the error as-is. Same as here:
https://github.com/hapijs/hapi-auth-hawk/blob/master/lib/index.js#L57-L60

I'll take a PR.

@StarpTech
Copy link
Contributor Author

Thanks!

@hueniverse hueniverse self-assigned this Jun 21, 2015
@hueniverse hueniverse added this to the 0.5.1 milestone Jun 21, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants