fix: coderabbit comments

Author: vanitha1822

src/main/java/com/iemr/tm/controller/foetalmonitor/FoetalMonitorController.java

@@ -50,7 +50,6 @@
 
 @RestController
 @RequestMapping(value = "/foetalMonitor", headers = "Authorization", consumes = "application/json", produces = "application/json")
-@PreAuthorize("hasRole('NURSE') || hasRole('DOCTOR') || hasRole('REGISTRAR')")
 public class FoetalMonitorController {
 	@Autowired
 	private FoetalMonitorService foetalMonitorService;

src/main/java/com/iemr/tm/utils/mapper/RoleAuthenticationFilter.java

@@ -51,12 +51,12 @@ protected void doFilterInternal(HttpServletRequest request, HttpServletResponse
 				filterChain.doFilter(request, response);
 				return;
 			}
-			Claims extractAllClaims = jwtUtil.extractAllClaims(jwtToken);
-			if(null == extractAllClaims) {
+			Claims claims = jwtUtil.validateToken(jwtToken);
+			if(null == claims) {
 				filterChain.doFilter(request, response);
 				return;
 			}
-			Object userIdObj = extractAllClaims.get("userId");
+			Object userIdObj = claims.get("userId");
 			String userId = userIdObj != null ? userIdObj.toString() : null;
 			if (null == userId || userId.trim().isEmpty()) {
 				filterChain.doFilter(request, response);

src/main/java/com/iemr/tm/utils/redis/RedisStorage.java

@@ -104,6 +104,7 @@ public void cacheUserRoles(Long userId, List<String> roles) {
 			 String key = "roles:" + userId;
 		        redisTemplate.delete(key); // Clear previous cache
 		        redisTemplate.opsForList().rightPushAll(key, roles);
+		        redisTemplate.expire(key, 30, java.util.concurrent.TimeUnit.MINUTES);
 		} catch (Exception e) {
 			logger.warn("Failed to cache role for user {} : {} ", userId, e.getMessage());
 		}