Ignorar métodos no Interceptor Vraptor

18 de janeiro de 2016 8 respostas Resolvido

sinkz 18 de janeiro de 2016

Estou tentando implementar autenticação com JWT está dando certo, porém o primeiro método que é responsável por criar o token tem que ser ignorado pelo Interceptor.

Criei uma anotação Public:

@Documented
    @Inherited
    @Retention(RetentionPolicy.RUNTIME)
    @Target({ElementType.METHOD, ElementType.TYPE})
    public @interface Public {
    }

E coloquei isso em meu Interceptor:

@Accepts
    public boolean accepts(ControllerMethod method) {
        return !method.containsAnnotation(Public.class);
    }

Porém só funciona quando faço a requisição por GET Se eu tento por POST a anotação e o Accepts não funcionam.

Ex:

@Public
    @Post
    @Path("/")
    @Consumes(value = "application/json", options = WithoutRoot.class)
    public void autenticar(Usuario usuario) {
        Usuario u = usuarioDAO.autenticar(usuario.getNome(), usuario.getSenha());
        if (u != null) {
            String token = JWTUtil.createToken(u.getId());
            result.use(Results.json()).from(token).serialize();
        } else {
            result.use(Results.status()).notAcceptable();
        }

    }

Como posso resolver esse problema?

8 Respostas

guivirtuoso 18 de jan. de 2016

Só uma dúvida…

Já que o recurso anotado com @Public não tem restrição nenhuma de acesso, porque você tá interceptando ele e depois fazendo um accept?

O ideial nao seria interceptar só métodos que teriam algum tipo de restrição … alguma anotação tipo @DevePossuirTokenValidoNoRequest por exemplo, e dali p/ frente realizar alguma validação p/ permitir ou não a continuidade do processo?

Att

sinkz 18 de jan. de 2016

Por isso que fiz o @Accepts. Pra não interceptar nada com essa anotação. Quando utilizo GET funciona, porém quando tento utilizar POST tenho esse erro: java.lang.IllegalStateException: token not set

guivirtuoso 18 de jan. de 2016

Mas quando você nao quer interceptar nada … é soh “não interceptar”

Como você está fazendo isso ?

Coloca a classe de interceptor ai …

Ou então da uma olhada lá no projeto que te passei… faço esse tipo de operção lá tbm.

https://github.com/guivirtuoso/app/blob/master/src/main/java/br/com/empresa/app/interceptors/PrecisaEstarLogadoInterceptor.java

sinkz 18 de jan. de 2016

Interceptador:

@Intercepts
@RequestScoped
public class SecurityInterceptor {

    private HttpServletRequest request;
    private Result result;

    public SecurityInterceptor() {
        this(null, null);
    }

    @Accepts
    public boolean accepts(ControllerMethod method) {
        return !method.containsAnnotation(Public.class);
    }

    
    @Inject
    public SecurityInterceptor(HttpServletRequest request, Result result) {
        this.request = request;
        this.result = result;
    }

    @AroundCall
    public void intercept(SimpleInterceptorStack stack) {
        System.out.println("Interceptando");
        String token = request.getHeader("authorization");

        Map<String, Object> claims;
        try {
            claims = JWTUtil.decode(token);

            Integer userId = (Integer) claims.get("user");
            if (userId != 1) {
                result.use(Results.http()).setStatusCode(401);
                result.use(Results.json()).from("This user does not exist", "msg").serialize();
            } else {
                result.use(Results.http()).addHeader("Authorization", token);

                stack.next();
            }
        } catch (InvalidKeyException | NoSuchAlgorithmException | IllegalStateException | SignatureException
                | IOException | JWTVerifyException e) {
            e.printStackTrace();
            result.use(Results.http()).setStatusCode(401);
            result.use(Results.json()).from(e.getMessage(), "msg").serialize();
        }
    }

}

guivirtuoso 18 de jan. de 2016

Será que ele tá fazendo alguma distinção entre “Authorization” e “authorization” ???

E outra… será que tem diferença entre Get ou Post quando se quer pegar os Headers da requisição?

Eu pesquisaria algo nessa linha…debugando e tal…

sinkz 18 de jan. de 2016

Beleza, vou debugar.

sinkz 18 de jan. de 2016

Acho que deu certo, coloquei a anotação que você passou: @AcceptsWithAnnotations e funcionou, kk tem alguma maneira de fazer uma negação nessa anotação? Exemplo: @AcceptsWithAnnotations(!Public.class) ?

Solucao aceita

guivirtuoso 18 de jan. de 2016 1 like

Cara, na minha opnião (que provavelmente deve estar errada ) , quando você não quer interceptar algo, basta não anotar, ai todo o fluxo seguirá normalmente.

Nesse caso ai do Publico… tipo… tudo já é publico.

Você quer apenas validar os endpoints que tem restrição de consumo…

Por isso acho que você está vendo a coisa um pouco invertida.

Ou então você pode ir vendo as implementações do @AcceptsWithAnnotations e criar a anotação @AcceptsWithoutAnnotations

Criado 18 de janeiro de 2016

Ultima resposta 18 de jan. de 2016

Respostas 8

Participantes 2

8 Respostas

Topicos relacionados