java.lang.IllegalStateException: getReader() bereits aufgerufen wurde für diese Anforderung

Sieht aus wie das restlet-framework genannt hat getRequestEntityStream() auf das Request-Objekt, das wiederum ruft getInputStream()so ruft getReader() auf die Anfrage wirft IllegalStateException. Die Servlet-API-Dokumentation für getReader() und getInputStream() sagt:

 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

Aus der Dokumentation es scheint, dass wir nicht anrufen können beide getReader() und getInputStream() des Request-Objekts. Ich schlage vor, Sie verwenden getInputStream() eher als getReader() in Ihre wrapper.

InformationsquelleAutor der Antwort Suresh Kumar

Soweit ich das sagen kann-servlets sind grundsätzlich gebrochen in dieser Hinsicht. Sie können versuchen, dieses problem zu umgehen, wie beschrieben hier aber andere Ursachen geheimnisvolle Probleme, wenn andere Dinge versuchen und mit ihm zu arbeiten.

Effektiv er schlägt vor, das Klonen der Wunsch, Lesen den Körper und dann in das geklonte Klasse überschreiben der getReader und getInputStream Methoden zum zurückgeben von dem Zeug bereits abgerufen.

Den code, mit dem ich endete, war diese:

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()));
    }
}

Sowieso das schien einwandfrei zu funktionieren, bis wir gemerkt haben, dass das hochladen einer Datei über den browser nicht funktioniert. Ich halbierter, durch die Veränderungen und entdeckt, das der übeltäter war.

Einige Leute in den Kommentaren in diesem Artikel sagen, müssen Sie das überschreiben von Methoden mit Parametern aber nicht erklären, wie dies zu tun.

Als Ergebnis habe ich überprüft, um zu sehen, ob es einen Unterschied zwischen den beiden Anfragen. Aber nach dem Klonen die Anfrage hatte es identische Sätze von Parametern (sowohl original-Anfrage + geklont hatte keine), sowie einen identischen Satz von Headern.

Jedoch in irgendeiner Art und Weise die Anfrage wurde durchgeführt und Schrauben ist das Verständnis der Anfrage weiter unten in der Zeile - in meinem Fall verursacht eine bizaare Fehler in einer Bibliothek (extdirectspring), wo etwas versucht hatte, Sie zu Lesen, den Inhalt als Json. Nehmen Sie sich den code, das Lesen der Körper in den filter gemacht, es funktioniert wieder.

Meine Berufung code sah so aus:

@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);
    }
}

Ich habe ausgefiltert den Inhalt ParseExtDirectTargetFrom aber es fordert getReader().

In meinem Fall war der filter funktioniert für alle anderen Anforderungen, aber das seltsame Verhalten in diesem Fall machte mir klar, etwas war nicht ziemlich Recht, und was ich versuche zu tun (umsetzen vernünftiges exception-handling, Verhalten, tests) nicht Wert, potenziell brechen zufällige künftige Anfragen (da ich nicht herausfinden konnte, was hatte die Anfrage an sich defekt).

Auch es ist erwähnenswert, dass der code gebrochen ist unvermeidlich - ich vermutet, es könnte etwas von Frühling, aber ServletRequest geht den ganzen Weg - das ist alles, was Sie bekommen, auch wenn Sie ein servlet, das von Grund auf neu, indem bilden von Unterklassen HttpServlet

Meine Empfehlung wäre das - nicht Lesen der request-body in einem filter. Sie werden die öffnung einer Dose Würmer, seltsame Probleme später auf.

InformationsquelleAutor der Antwort Jonny Leeds