Skip to content

Commit 8bdcb83

Browse files
♻️ refactoring code
1 parent 62643a7 commit 8bdcb83

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

src/main/java/org/rrajesh1979/utils/JWTUtil.java

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,9 @@
2020
import java.util.HashMap;
2121
import java.util.Map;
2222
import java.util.Objects;
23+
import java.util.TreeMap;
2324

25+
import com.sun.source.tree.Tree;
2426
import lombok.extern.slf4j.Slf4j;
2527
import org.javatuples.Pair;
2628
import org.json.JSONObject;
@@ -54,9 +56,10 @@ public static Pair<String, String> createJWT(String typ, String alg, String user
5456
String iss, String sub, String aud, boolean iat, long exp) {
5557

5658
/* Construct JWT Header */
57-
Map<String, Object> header = new HashMap<>();
58-
header.put("typ", Objects.requireNonNullElse(typ, "JWT"));
59+
Map<String, Object> header = new TreeMap<>();
5960
header.put("alg", Objects.requireNonNullElse(alg, "HS256"));
61+
header.put("typ", Objects.requireNonNullElse(typ, "JWT"));
62+
log.debug(header.toString());
6063

6164
/* Construct JWT Payload */
6265
JSONObject payload = new JSONObject(userInput);

src/test/java/org/rrajesh1979/utils/JWTUtilTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@ void encodeDecodeJWTTest() {
3838
String key = jwtAndKey.getValue1();
3939

4040
Pair<String, String> decodedJwtAndKey = JWTUtil.decodeJWT(jwt, key);
41-
assertEquals(decodedJwtAndKey.getValue0(), "{typ=JWT, alg=HS512}");
42-
assertEquals(decodedJwtAndKey.getValue1(), "{sub=JWT Encoder, aud=Hello JWT, name=Joe, iss=rrajesh1979, picture=https://example.com/image.png}");
41+
assertEquals("{alg=HS512, typ=JWT}", decodedJwtAndKey.getValue0());
42+
assertEquals("{sub=JWT Encoder, aud=Hello JWT, name=Joe, iss=rrajesh1979, picture=https://example.com/image.png}", decodedJwtAndKey.getValue1());
4343
}
4444

4545
@Test

0 commit comments

Comments
 (0)