Skip to content
This repository has been archived by the owner on Mar 14, 2022. It is now read-only.

Encode the issuer in a JWT #392

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Encode the issuer in a JWT #392

wants to merge 1 commit into from

Conversation

jcoyne
Copy link
Contributor

@jcoyne jcoyne commented Apr 10, 2018

This allows us to track which upstream application is using the TACO
service.

Presenlty, go-swagger gives us no options for making sure that two
authorization tokens are passed, so by using JWT, we encode the subject
(user identification) and issuer (service) in a single token.

Fixes #375

Copy link
Contributor

@aaron-collier aaron-collier left a comment

Choose a reason for hiding this comment

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

This command appears to be invalid --subject should be --sub, I get an error with --subject

docs/tokens.md Outdated
*note* the secret must be the same secret that TACO is configured with
(in the `TACO_SECRET_KEY` variable)
```
jwt encode --secret superSeckeret --iss hydrus.stanford.edu --subject "lmcrae@stanford.edu"
Copy link
Contributor

Choose a reason for hiding this comment

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

This command appears to be invalid --subject should be --sub, I get an error with --subject

@aaron-collier
Copy link
Contributor

I'm also just getting 401 unauthenticated errors when following the readme and token.md information here.

This allows us to track which upstream application is using the TACO
service.

Presenlty, go-swagger gives us no options for making sure that two
authorization tokens are passed, so by using JWT, we encode the subject
(user identification) and issuer (service) in a single token.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants