Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Broken header serialization in v0.4.4

See original GitHub issue

With version 0.4.4 the JwtHeader is wrongly trasformed as

[{"alg":"HS256"},{"typ":"JWT"}]

instead of

{"alg":"HS256","typ":"JWT"}

This is because in the JwtHeader class, in the asJsonString function we do:

    implicit val formats = org.json4s.DefaultFormats
    org.json4s.jackson.Serialization.write(toSerialize)

instead of

SimpleJsonSerializer(toSerialize)

Please please fix it as this is making impossible to use the library!

Cheers, D.

Issue Analytics

State:
Created 7 years ago
Comments:5 (5 by maintainers)

Top GitHub Comments

1reaction

jasongoodwincommented, Jan 11, 2017

Hey thanks, I removed the SimpleJsonLibrary as it may be vulnerable to injection attacks. I’ll look at this right now.

0reactions

jasongoodwincommented, Jan 11, 2017

0.4.5 published - it looks good.

Top Results From Across the Web

kubernetes/ingress-nginx - DO Proxy Protocol broken header

Is this a BUG REPORT or FEATURE REQUEST? (choose one): BUG REPORT NGINX Ingress controller version: 0.24.0 Kubernetes version (use kubectl ...

json_serializable | Dart Package - Pub.dev

Require json_annotation '>=4.4.0 <4.5.0' . ... NOTE: This is a potential breaking change if any user code relies on the previously generated private ......

Serialization breaks in .NET 4.5 - Stack Overflow

In 4.5, the implementation of XmlSerializer was replaced with one that isn't dependent on the C# compiler. While it provides better startup ...

Firebase Apple SDK Release Notes - Google

Breaking change: import Firebase will no longer implicitly import Firebase ... Updated Analytics public header documentation to reference Swift names.

Kong Gateway Changelog | Kong Docs

Fixed an issue where queue ID serialization did not include queue_size and ... Fixed an issue with accept headers not being split and...