Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BIG-16 #close - implement token verification function #11

Merged
merged 11 commits into from
Jan 11, 2018

Conversation

smulube
Copy link
Contributor

@smulube smulube commented Jan 10, 2018

The meat of this PR is in the ValidateToken method which uses a JWT library from square to parse and validate tokens where a valid token is: a) signed by the provider's secret, and b) has not expired. The library ensures a valid signing algorithm is used I think.

Other than that all that is in this pr is using the github.com/pkg/errors library to wrap errors

Sam Mulube added 5 commits January 5, 2018 10:18
Really there is nothing here but a vague function definition.
* Wrap errors in provider
* Provide implementation for ValidateToken method
* Make provider test external again (i.e. test from `bigiot_test` package.
Copy link

@chespinoza chespinoza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Sam Mulube and others added 6 commits January 10, 2018 17:50
This allows re-activating an existing offering without having to send the full description everytime
get back from git if required
This was just added to help me build the string, but should be deleted
Remove some commented out code blocks
BIG-32 #close Activate offering
@smulube smulube merged commit 250f538 into master Jan 11, 2018
@smulube smulube deleted the access_token_verification branch January 11, 2018 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants