Skip to content

Commit 87287ea

Browse files
committed
FIX: RenewTokenFilter
1 parent badb8aa commit 87287ea

2 files changed

Lines changed: 15 additions & 25 deletions

File tree

src/main/java/io/github/isagroup/filters/RenewTokenFilter.java

Lines changed: 15 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -36,31 +36,22 @@ public class RenewTokenFilter extends OncePerRequestFilter {
3636
protected void doFilterInternal(HttpServletRequest request, HttpServletResponse response, FilterChain filterChain)
3737
throws ServletException, IOException {
3838

39-
try {
40-
String pricingJwt = parsePricingJwt(request);
41-
String authJwt = parseAuthJwt(request);
42-
43-
if (authJwt != null && jwtUtils.validateJwtToken(authJwt) && pricingContext.userAffectedByPricing()) {
44-
45-
String newToken = pricingEvaluatorUtil.generateUserToken();
46-
47-
Map<String, Map<String, Object>> newTokenFeatures = jwtUtils.getFeaturesFromJwtToken(newToken);
48-
Map<String, Map<String, Object>> jwtFeatures = jwtUtils.getFeaturesFromJwtToken(pricingJwt);
49-
50-
String newTokenFeaturesString = "";
51-
String jwtFeaturesString = "";
52-
53-
if (newTokenFeatures != null) newTokenFeaturesString = newTokenFeatures.toString();
54-
55-
if (jwtFeatures != null) jwtFeaturesString = jwtFeatures.toString();
56-
57-
if (!newTokenFeaturesString.equals(jwtFeaturesString)) {
58-
response.addHeader("Pricing-Token", newToken);
59-
}
39+
String pricingJwt = parsePricingJwt(request);
40+
String authJwt = parseAuthJwt(request);
41+
42+
if (authJwt != null && jwtUtils.validateJwtToken(authJwt) && pricingContext.userAffectedByPricing()) {
43+
44+
String newToken = pricingEvaluatorUtil.generateUserToken();
45+
46+
String newTokenFeatures = jwtUtils.getFeaturesFromJwtToken(newToken).toString();
47+
String jwtFeatures = "";
48+
49+
if (pricingJwt != null && !pricingJwt.equals("null")) jwtFeatures = jwtUtils.getFeaturesFromJwtToken(pricingJwt).toString();
50+
51+
if (!newTokenFeatures.equals(jwtFeatures)) {
52+
response.addHeader("Pricing-Token", newToken);
6053
}
61-
} catch (Exception e) {
62-
logger.error("Cannot set user authentication: {}", e);
63-
logger.info("Anonymous user logged");
54+
6455
}
6556

6657
filterChain.doFilter(request, response);

src/main/java/io/github/isagroup/services/jwt/PricingJwtUtils.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ public String getSubjectFromJwtToken(String token) {
5252
*/
5353
public String generateTokenFromUsername(String username) {
5454
Map<String, Object> claims = new HashMap<>();
55-
claims.put("authorities", pricingContext.getUserAuthorities());
5655
return Jwts.builder().setClaims(claims).setSubject(username).setIssuedAt(new Date())
5756
.setExpiration(new Date((new Date()).getTime() + pricingContext.getJwtExpiration()))
5857
.signWith(SignatureAlgorithm.HS512, pricingContext.getJwtSecret()).compact();

0 commit comments

Comments
 (0)