Блог

У меня есть объект ответа, который я хочу вернуть пользователю, если в контроллере возникнет исключение. Однако, когда я пытаюсь отправить обратно BadRequest, я, похоже, не могу отправить обратно свой объект ответа. Итак, мой вопрос в том, как мне отредактировать BadRequst, чтобы он содержал мой объект ответа и / или как мне отправить обратно мой объект ответа с кодом состояния ошибки?

Контроллер

 public async Task<IActionResult> Login([FromBody] LoginViewModel model) {
    Response response = new Response(); 
    try {
        if (ModelState.IsValid) {
            var result = await signInManager.PasswordSignInAsync(model.Email, model.Password, model.RememberMe, lockoutOnFailure: false);
            if (result.Succeeded) {
                logger.LogInformation(1, "User logged in.");
                return Json("User logged in.");
            }
            if (result.RequiresTwoFactor) {
                response.id = 1 ; 
                throw new LoginException("Login requiest two factor", new InvalidOperationException());
            }
            if (result.IsLockedOut) {
                response.id = 2 ; 
                logger.LogWarning(2, "User account locked out.");
                throw new LoginException("User account locked out", new InvalidOperationException());
            }
            else {
                response.id = 3 ; 
                throw new LoginException("Invalid login attempt", new InvalidOperationException());
            }
        }
        response.id = 4 ; 
        var modelErrors = ModelState.Values.ToList(); 
        throw new LoginException("Model State Error", ModelState, new InvalidOperationException());
    } catch (LoginException ex){
        return BadRequest(response); // response with status: 400 Bad Request for URL, no response object is in here. 
    }
}
  

Интерфейс

 login(email:any, password:any, remember:any){
    //let body:User = {email:email, password:password}; 
    let headers = new Headers({ 'Content-Type': 'application/json' });

    let body = {Email:email, Password:password,RememberMe:false }; 
    console.log(body); 
    this.http.post('/api/Account/Login', body ,{headers:headers})
    .map(response  => response.json())
    .subscribe(
         response => { 
             console.log("Success !!!:n" response); 
             this.router.navigate(['/home']);
        }, 
        error => {
             console.log("Error !!!:n" error); 
        }
    );
}
  

Не совсем понятно, о чем вы спрашиваете?

Вы хотите вернуть только код состояния 400? Используйте версию без параметров return BadRequest()

Хотите другой код состояния? Используйте другой метод. BadRequest имеет имя с помощью HTTP Statuscode 400 (BadRequest). Если вы хотите самостоятельно установить код состояния, используйте return Status(HttpStatusCode.Unauthorized); or что угодно.

Хотите вернуть сообщение об ошибке в формате json? Для этого используйте классы ошибок или анонимный класс:

 return BadRequest(new { ErrorMessage = "Account is locked." });
  

Но что бы вы ни хотели сделать, выбрасывать исключения в контроллере неправильно. Действие контроллера НИКОГДА не должно вызывать исключение. Также, как указано в комментариях, исключения должны использоваться для исключительных случаев. Неправильный пароль, заблокированный пользователь или неправильный метод аутентификации (т. Е. Отсутствие двухфакторной аутентификации, когда это требуется) являются ожидаемыми ошибками и не должны обрабатываться с помощью исключений.

Исключения (только когда они генерируются) по своей сути являются дорогостоящими операциями в компьютерной программе. По этой причине Microsoft внедрила IdentityResult класс для возврата списка ожидаемых ошибок, а не для создания исключений во время вызова signInManager.PasswordSignInAsync

Вместо возврата IActionResult , почему бы вам не вернуть a HttpResponseMessage , а затем вы можете сделать что-то вроде: return Request.CreateResponse(System.Net.HttpStatusCode.BadRequest)