Rückgabe HTTP Statuscode 204 auf null mit spring @RestController

Dieser gibt 200 OK Content-Length: 0

@RestController
public class RepoController {
    @RequestMapping(value = "/document/{id}", method = RequestMethod.GET)
    public Object getDocument(@PathVariable long id) {
       return null;
    }

}

Einfach gesagt ich möchte es zurückgeben 204 No Content auf null.

Gibt es eine Möglichkeit zu zwingen, spring-mvc/rest zurück 204 auf null-nicht 200? Ich will nicht zu ändern, jede rest-Methode zurück ResponseEntity oder sowas, nur die Karte null 204

InformationsquelleAutor user1606576 | 2015-09-04

Natürlich, ja.

Option 1 :

@RestController
public class RepoController {
    @RequestMapping(value = "/document/{id}", method = RequestMethod.GET)
    public Object getDocument(@PathVariable long id, HttpServletResponse response) {
       Object object = getObject();
       if( null == object ){
          response.setStatus( HttpStatus.SC_NO_CONTENT);
       }
       return object ;
    }
}

Option 2 :

@RestController
public class RepoController {
    @RequestMapping(value = "/document/{id}", method = RequestMethod.GET)
    public Object getDocument(@PathVariable long id) {
       Object object = getObject();
       if ( null == object ){
          return new ResponseEntity<Void>(HttpStatus.NO_CONTENT);
       }

       return object ;
    }
}

Haben könnte Tippfehler, aber Sie bekommen die Idee.

Option 2 für mich gearbeitet. Schöne Lösung!
Leider wird keine dieser Optionen ist eine gute (saubere) Lösung. Idealerweise ist Ihr controller-Methode liefert einen typisierten Objekt eher als Object oder ResponseEntity, und es wäre eine Art von intercepter, die bemerken würde, dass Sie null zurückgeben, und legen Sie die NO_CONTENT status-code (oder war ich einfach verwöhnt von Jersey und RESTEasy).

InformationsquelleAutor Karthik R

29

Können Sie die @ResponseStatus annotation. Auf diese Weise können Sie eine void-Methode, und Sie nicht haben, um zu bauen ein ResponseEntity.
```
@DeleteMapping(value = HERO_MAPPING)
@ResponseStatus(value = HttpStatus.NO_CONTENT)
public void delete(@PathVariable Long heroId) {
    heroService.delete(heroId);
}
```
BTW Rückkehr 200, wenn das Objekt vorhanden ist und 204 ansonsten ist es etwas ungewöhnlich bezüglich API-REST-design. Es ist üblich, erzeugen einen 404 (nicht gefunden), wenn das angeforderte Objekt nicht gefunden. Und dies kann erreicht werden durch eine ControllerAdvice.

Im Frühjahr RUHE, es ist besser, die Ausnahmen behandeln mit einem Exception-handler, anstatt die Logik zu entscheiden, der Antwort-status, etc. Dies ist ein Beispiel zur Verwendung der @ControllerAdvice Anmerkung: http://www.jcombat.com/spring/exception-handling-in-spring-restful-web-service

Ich denke, der OP wollte schon wieder 200 auf Erfolg, und 204 wenn er null. Sie können alles, was Sie wollen, in ResponseStatus. Aber die Idee war es, mehrere mögliche HTTP-codes für einen Endpunkt.
Ich Stimme mit @Jean-FrançoisBeauchef. Dies ist nicht der richtige Umgang. Diese Antwort status angewendet wird, unabhängig vom Ergebnis (ohne Ausnahme-Szenario). Der Fragesteller wollte wissen, wie gerecht Zustand von null
Sorry, aber ich bin nicht einverstanden. Bei der Verwendung von Spring REST, es ist besser, die Behandlung von Ausnahmen und Antwort-codes mit einem Exception-handler. Dies ist ein Beispiel jcombat.com/spring/...
ja, natürlich. Sollten Sie eine exception-handler, um eine Ausnahme behandeln. Aber das ist nicht das, was der OP fragte. Er wollte schon wieder 200 auf einer richtigen Antwort mit dem Inhalt, und 204 (das ist auch eine richtige Antwort), wenn es keinen Inhalt (oder null, wenn Sie bevorzugen). Die OP gar nicht erwähnen Ausnahmen. Und in Java (Spring oder anderweitig), Ausnahmen sollten nicht verwendet werden zur Steuerung des Datenflusses.
Blick auf die Antworten von mahieus und Marek Raszewski. Sie werden verstehen, was der OP war für das Schießen.

InformationsquelleAutor spekdrum

Löste ich dieses problem mit einem filter. Es ist global und einfach.

package your.package.filter;

import org.springframework.http.HttpStatus;
import org.springframework.web.filter.OncePerRequestFilter;

import javax.servlet.*;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import java.io.IOException;

public class NoContentFilter extends OncePerRequestFilter {

    @Override
    protected void doFilterInternal(HttpServletRequest httpServletRequest, HttpServletResponse httpServletResponse, FilterChain filterChain) throws ServletException, IOException {
        filterChain.doFilter(httpServletRequest, httpServletResponse);
        if (httpServletResponse.getContentType() == null ||
                httpServletResponse.getContentType().equals("")) {
            httpServletResponse.setStatus(HttpStatus.NO_CONTENT.value());
        }
    }
}

und fügen Sie den folgenden in Ihre web.xml

<filter>
    <filter-name>restNoContentFilter</filter-name>
    <filter-class>your.package.filter.NoContentFilter</filter-class>
</filter>

<filter-mapping>
    <filter-name>restNoContentFilter</filter-name>
    <url-pattern>/rest/*</url-pattern>
</filter-mapping>

(HttpStatus.NO_CONTENT.Wert ()), die ich kopieren-einfügen die funktionierende Lösung +1 Stimme
Sehr schöne Lösung

InformationsquelleAutor mahieus

Können Sie versuchen, diese :

@RestController
public class RepoController {

    @RequestMapping(value = "/document/{id}", method = RequestMethod.GET)
    public ResponseEntity<String> getDocument(@PathVariable long id) {

       if(noError) {
           ............
           return new ResponseEntity<String>(HttpStatus.OK); 
       }
       else {
           return new ResponseEntity<String>(HttpStatus.BAD_REQUEST);
       }
   }
}

Uou ändern müssen, HttpStatus.BAD_REQUEST mit dem äquivalent für 204 code status

InformationsquelleAutor Bilal BBB

Gleiche Antwort aber gelöst durch AOP:

@Aspect
public class NoContent204HandlerAspect {

  @Pointcut("execution(public * xx.xxxx.controllers.*.*(..))")
  private void anyControllerMethod() {
  }

  @Around("anyControllerMethod()")
  public Object handleException(ProceedingJoinPoint joinPoint) throws Throwable {

    Object[] args = joinPoint.getArgs();

    Optional<HttpServletResponse> response = Arrays.asList(args).stream().filter(x -> x instanceof HttpServletResponse).map(x -> (HttpServletResponse)x).findFirst();

    if (!response.isPresent())
      return joinPoint.proceed();

    Object retVal = joinPoint.proceed();
    if (retVal == null)
      response.get().setStatus(HttpStatus.NO_CONTENT.value());

    return retVal;
  }
}

Können Sie mehr Details auf, was ist eigentlich hier passiert. Ich bin zu versuchen, eine ähnliche Lösung. Meine Idee ist, automatisch wieder zurück 204 für alle Methoden, die einen void-Rückgabetyp. Es scheint, wie diese Lösung erfordert die Methode eine HttpServletResponse, die als ein Eingang. Ich möchte vermeiden, dass.
Versuchen Sie, um eine statische Methode zum abrufen HttpServletResponse wie here Der rest sollte gleich sein aber der @Pointcut erstrecken sollte, die nur leere Methoden.

InformationsquelleAutor Marek Raszewski

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.