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.

Sign up for GitHub

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

Jump to bottom

Ensure that claim content is json serializable #44

Closed
lcobucci opened this issue Oct 5, 2015 · 1 comment
Closed

Ensure that claim content is json serializable #44

lcobucci opened this issue Oct 5, 2015 · 1 comment
Assignees
@lcobucci
Labels
BC-break Improvement Won't Fix

Comments

@lcobucci
Copy link
Owner

lcobucci commented Oct 5, 2015

No description provided.

@lcobucci lcobucci added this to the 4.0.0 milestone Oct 5, 2015
@MarijnKoesen MarijnKoesen mentioned this issue Sep 30, 2016
Closed
@lcobucci lcobucci mentioned this issue Oct 24, 2016
Merged
@lcobucci lcobucci removed this from the 4.0.0-alpha1 milestone Feb 27, 2019
@lcobucci
Copy link
Owner Author

lcobucci commented Nov 9, 2020

I've decided not to do this. I mean, people have been using this library for years already and haven't reported issues. Encoding the claim and decoding just to validate stuff seems overkill for this library.

@lcobucci lcobucci closed this as completed Nov 9, 2020
@lcobucci lcobucci self-assigned this Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lcobucci lcobucci

Labels
BC-break Improvement Won't Fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lcobucci