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

Fixed command line encoding #128

Merged
merged 1 commit into from
Apr 15, 2015
Merged

Conversation

nielsdraaisma
Copy link
Contributor

I tried running the example invocation
jwt --key=secret iss=me exp=1302049071

This failed and returned an error "Not enough segments" followed by a sys.exit(1).
It looks like it tried to parse the encoding parameters and JWT token data, which obviously fails.

I've worked around it and the example scenario's work,but may be a --encode/--decode parameter would be more explicit.

@jpadilla
Copy link
Owner

jpadilla commented Apr 4, 2015

Ugh this thing. I think it either needs to go or get rebuilt. Plus it needs some kind of tests.

@nielsdraaisma do you use it for anything significant/important?

@nielsdraaisma
Copy link
Contributor Author

Don't use it at all, came across it looking for something that could generate a token, and that was broken.

@mark-adams
Copy link
Contributor

@jpadilla Are we wanting to keep this? Is it useful?

@jpadilla
Copy link
Owner

jpadilla commented Apr 8, 2015

@mark-adams blah I'd be happier without it really. What would the best "semver" way of getting rid of it soon?

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

3 participants