Estoy trabajando con JWT y es necesario manejar los claims desde una clase, para esto cree la siguiente AuthContextHolder.java
:
package com.payulatam.ppp4.secure.api.controllers;
import javax.servlet.http.HttpServletRequest;
import org.springframework.web.context.request.RequestContextHolder;
import org.springframework.web.context.request.ServletRequestAttributes;
import com.payulatam.ppp4.secure.api.model.commons.SecureApiResources;
import io.jsonwebtoken.Claims;
public class AuthContextHolder {
public Integer getWebUserId(){
return getIntegerClaims(SecureApiResources.WEB_USER_ID_PARAM);
}
public Integer getMerchantId(){
return getIntegerClaims(SecureApiResources.MERCHANT_ID_PARAM);
}
public Integer getDefaultAccountId(){
return getIntegerClaims(SecureApiResources.DEFAULT_ACCOUNT_ID_PARAM);
}
public Integer getIntegerClaims(String claimParameter){
HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
final Claims claims = (Claims) request.getAttribute(SecureApiResources.CLAIMS_ATTR);
return (Integer) claims.get(claimParameter);
}
}
Perso me surge la duda de si esta clase puede mejorarse a través del uso de un patrón o con un enfoque distinto.
A continuación muestro un ejemplo de la implementación de esta clase:
private AuthContextHolder authContextHolder = new AuthContextHolder();
@RequestMapping(value = SecureApiResources.Asobancaria.URL_ASOBANCARIA_MERCHANT_ACCOUNTS, method = RequestMethod.GET, produces = "application/json")
public @ResponseBody List<AccountAsobancariaBank> getAccountsAsobancaria() throws Exception {
List<AccountAsobancariaBank> asobancariaData = asobancariaService.getAccounts(authContextHolder.getMerchantId());
if ((asobancariaData.get(0) == null)) {
HttpErrors httpErrors = new HttpErrors(
SecureApiResources.ERROR_ACCOUNTS_NOT_FOUND);
throw new EntityNotFoundException(httpErrors);
}
return asobancariaData;
}
Agradezco sus consejos.
AuthContextHolder
comoprivate webUserId
y se utilicen los metodos como getters publicos. Si está bien que esta sea una clase simple.AuthContextHolder
necesita ser creada/recreada por request, entonces no es relevante que mantenga estado. Por último, solo realiza la optimización que indicas si puedes demostrar que el métodogetIntegerClaims
realmente es un cuello de botella en la aplicación vía micro benchmarks o vía ejecutar la aplicación y colocar un profiler para medir el rendimiento del método bajo situaciones de estrés.