Skip to content

Commit d8fe9a2

Browse files
authored
Empty string audience claim should be deserialized as empty string (#663)
1 parent 923e9c4 commit d8fe9a2

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

lib/src/main/java/com/auth0/jwt/impl/PayloadDeserializer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ List<String> getStringOrArray(ObjectCodec codec, Map<String, JsonNode> tree, Str
5454
if (node == null || node.isNull() || !(node.isArray() || node.isTextual())) {
5555
return null;
5656
}
57-
if (node.isTextual() && !node.asText().isEmpty()) {
57+
if (node.isTextual()) {
5858
return Collections.singletonList(node.asText());
5959
}
6060

lib/src/test/java/com/auth0/jwt/impl/PayloadDeserializerTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -145,14 +145,14 @@ public void shouldGetStringArrayWhenParsingTextNode() {
145145
}
146146

147147
@Test
148-
public void shouldGetEmptyStringArrayWhenParsingEmptyTextNode() {
148+
public void shouldGetEmptyStringInArrayWhenParsingEmptyTextNode() {
149149
Map<String, JsonNode> tree = new HashMap<>();
150150
TextNode textNode = new TextNode("");
151151
tree.put("key", textNode);
152152

153153
List<String> values = deserializer.getStringOrArray(objectMapper, tree, "key");
154154
assertThat(values, is(notNullValue()));
155-
assertThat(values, is(IsEmptyCollection.empty()));
155+
assertThat(values, is(IsIterableContaining.hasItem("")));
156156
}
157157

158158
@Test

0 commit comments

Comments
 (0)