java.lang.IllegalStateException: getReader() a déjà été appelée pour cette demande

Ressemble à la restlet framework a appelé getRequestEntityStream() sur l'objet de la Requête qui appelle à son tour getInputStream(), afin de l'appelant getReader() sur la demande jette IllegalStateException. La documentation de l'API Servlet pour getReader() et getInputStream() dit:

 public java.io.BufferedReader getReader()
    ...
    ...
Throws:
    java.lang.IllegalStateException - if getInputStream() method has been called on this request

 public ServletInputStream getInputStream()
    ...
    ...
    Throws:
    java.lang.IllegalStateException - if the getReader() method has already been called for this request

De la documentation, il semble que l'on ne peut pas appeler à la fois getReader() et getInputStream() sur l'objet de la Requête. Je vous suggère d'utiliser getInputStream() plutôt que getReader() dans votre enveloppe.

OriginalL'auteur

Aussi loin que je peux dire que les servlets sont fondamentalement rompu à cet égard. Vous pouvez essayer de contourner ce problème comme indiqué ici mais qui provoque d'autres mystérieux des problèmes lorsque d'autres choses à essayer et travailler avec elle.

Effectivement, il suggère le clonage de la demande, la lecture du corps et ensuite dans le clonés de classe substitution de la getReader et getInputStream méthodes de retourner les choses déjà récupérées.

Le code que j'ai fini avec cela:

import javax.servlet.ServletInputStream;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletRequestWrapper;
import java.io.*;

//this class stops reading the request payload twice causing an exception
public class WrappedRequest extends HttpServletRequestWrapper
{
    private String _body;
    private HttpServletRequest _request;

    public WrappedRequest(HttpServletRequest request) throws IOException
    {
        super(request);
        _request = request;

        _body = "";
        try (BufferedReader bufferedReader = request.getReader())
        {
            String line;
            while ((line = bufferedReader.readLine()) != null)
                _body += line;
        }
    }

    @Override
    public ServletInputStream getInputStream() throws IOException
    {
        final ByteArrayInputStream byteArrayInputStream = new ByteArrayInputStream(_body.getBytes());
        return new ServletInputStream()
        {
            public int read() throws IOException
            {
                return byteArrayInputStream.read();
            }
        };
    }

    @Override
    public BufferedReader getReader() throws IOException
    {
        return new BufferedReader(new InputStreamReader(this.getInputStream()));
    }
}

De toute façon, cela semble bien fonctionner jusqu'à ce que nous nous sommes rendu compte que le téléchargement d'un fichier depuis le navigateur n'a pas de travail. Je l'ai coupée en deux par les changements et découvert que c'était le coupable.

Certaines personnes dans les commentaires que dans l'article de dire vous avez besoin de remplacer les méthodes à faire avec les paramètres mais n'expliquent pas comment le faire.

Par conséquent, j'ai vérifié pour voir si il n'y avait aucune différence entre les deux demandes. Cependant, après le clonage de la demande qu'il avait identiques jeux de paramètres (à la fois de la demande d'origine + cloné n'y en avait aucun) aswell comme identique d'un jeu d'en-têtes.

Cependant d'une certaine manière, la demande a été effectuée et le vissage jusqu'à la compréhension de la demande par la suite - dans mon cas, provoquant une bizaare erreur dans une bibliothèque (extdirectspring) où quelque chose a été d'essayer de lire le contenu Json. Prendre le code qui lisent le corps dans le filtre de le faire fonctionner à nouveau.

Mon code d'appel qui ressemblait à ceci:

@Override
public void doFilter(ServletRequest request, ServletResponse response, FilterChain filterChain) throws IOException, ServletException
{
    HttpServletRequest properRequest = ((HttpServletRequest)request);

    String pathInfo = properRequest.getPathInfo();
    String target = "";
    if(pathInfo == null)
        pathInfo = "";

    if(pathInfo.equals("/router"))
    {
        //note this is because servlet requests hate you!
        //if you read their contents more than once then they throw an exception so we need to do some madness
        //to make this not the case
        WrappedRequest wrappedRequest = new WrappedRequest(properRequest);
        target = ParseExtDirectTargetFrom(wrappedRequest);
        request = wrappedRequest;
    }

    boolean callingSpecialResetMethod = pathInfo.equals("/resetErrorState") || target.equals("resetErrorState");
    if(_errorHandler.IsRejectingRequests() && !callingSpecialResetMethod)
        return;

    try {
        filterChain.doFilter(request, response);
    }
    catch (Exception exception) {
        ((HttpServletResponse) response).sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, "ERROR");
        _errorHandler.NotifyOf(exception);
    }
}

J'ai oublié le contenu de ParseExtDirectTargetFrom mais il appelle getReader().

Dans mon cas, le filtre a été de travail pour toutes les autres demandes, mais le comportement étrange dans ce cas, m'a fait réaliser quelque chose n'allait pas et que j'essaye de faire (mettre en œuvre judicieuse de gestion des exceptions de comportement pour les tests) n'était pas la peine de ce qui peut potentiellement rendre aléatoire les futures demandes (que je ne comprenais pas ce qui avait causé la demande pour devenir cassé).

Aussi, il est intéressant de noter que le code cassé est inévitable - je suppose qu'il pourrait être quelque chose de printemps, mais ServletRequest va tout le chemin jusqu' - c'est tout ce que vous obtenir, même si vous faisiez une servlet à partir de zéro par sous-classement HttpServlet

Ma recommandation serait ce - ne lisent pas le corps de la requête dans un filtre. Vous allez ouvrir une boîte de pandore qui va provoquer d'étranges problèmes plus tard.

OriginalL'auteur