ASP.NET .NET Core Web Api -- почему параметры всегда null?

Question

[DailyDDose]

Что я делаю не так?

AuthController.cs

using Microsoft.AspNetCore.Mvc;
 
namespace WebApiServer.Controllers
{
    [Route("api/[controller]")]
    public class AuthController : Controller
    {
        // POST api/auth
        [HttpPost]
        public string Post([FromBody] string login, [FromBody] string pass)
        {
            System.Console.WriteLine("login: " + login);
            System.Console.WriteLine("pass: " + pass);
            if (
                (login == "dailydose" && pass == "iuwqfheiwqnef")
                ||
                (login == "admin" && pass == "iuwqfheiwqnef")
            )
            {
                return "SUCCESS";
            }
 
            return "ERROR";
        }
    }
}

Answer 1

[eRKa]

Атрибут FromBody может применяться только к одному параметру. Он нужен, когда Вы в теле передаете параметр простого типа. Если Вам нужно отправить несколько параметров пост запросом, сделайте сложный тип (dto, viewModel, как нравится)

public class AuthorizeModel
{
	public string Login { get; set; }
	public string Password { get; set; }
}

В этом случае атрибут можно не указывать.
Вот тут подробности.

Web API reads the response body at most once, so only one parameter of an action can come from the request body. If you need to get multiple values from the request body, define a complex type.

Comments

[DailyDDose]

Хм, странно. Модель так же забивается null'ами.

[DailyDDose]

eRKa всё же аттрибут FromBody нужен.
С ним заработало :)

[eRKa]

DailyDDose, пока стоит брейкпоинт, зайдите через Locals в this.Request.Body и посмотрите, вообще пришли ли параметры.
И сдается мне, что в постмене надо application/x-www-form-urlencoded указать, а не raw

[eRKa]

DailyDDose, не в курсе почему именно так у Вас, но у меня работает и без него) Запросы идут аяксом со страницы.

[DailyDDose]

eRKa,
Да уж... Что-то совсем перестали данные приходить, опять одни null.
Как с аттрибуттом FromBody, так и без него -- null.

[eRKa]

DailyDDose, дружище, разберись с контент тайпом запросов и соответствующими им способами байдинга параметров.
Например, если будешь отправлять через x-www-form-urlencoded (по умолчанию аякс так отправляет), то атрибут не нужен. Если хочешь через jsontype, то нужно указать FromBody.
У меня все работает.

Там может быть два варианта: либо не пришли параметры, либо они не распарсились из-за неправильно указанного байдинга. Поэтому я и говорю, что на брекпоинте посмотрите в реквесте где параметры и какие, и уже от этого плясать.

[DailyDDose]

eRKa, ой, боже... Как же всё сложно, хоть обратно на пэхэпэ возвращайся, ей богу!

Попробовал я значит вашими ояксами от $ достучаться до сервака.
Они вроде бы даже теперь приходят, но вот как их теперь вытащить?
Провайдер/сервис/миддлвару надо подключать какую?

using Microsoft.AspNetCore;
using Microsoft.AspNetCore.Hosting;

namespace WebApiServer
{
    public class Program
    {
        public static void Main(string[] args)
        {
            BuildWebHost(args).Run();
        }

        public static IWebHost BuildWebHost(string[] args) =>
            WebHost.CreateDefaultBuilder(args)
                .UseStartup<Startup>()
                .Build();
    }
}

using Microsoft.AspNetCore.Builder;
using Microsoft.AspNetCore.Hosting;
using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.DependencyInjection;

namespace WebApiServer
{
    public class Startup
    {
        public Startup(IConfiguration configuration)
        {
            Configuration = configuration;
        }

        public IConfiguration Configuration { get; }

        // This method gets called by the runtime. Use this method to add services to the container.
        public void ConfigureServices(IServiceCollection services)
        {
            services.AddMvc();
        }

        // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
        public void Configure(IApplicationBuilder app, IHostingEnvironment env)
        {
            app.UseDefaultFiles();
            app.UseStaticFiles();
            
            if (env.IsDevelopment())
            {
                app.UseDeveloperExceptionPage();
            }

            app.UseMvc();
        }
    }
}

[eRKa]

DailyDDose, Для того, чтобы получить параметры в контроллере из запроса в большинстве случаев ничего специально делать не надо. Для каких то исключительных случаев есть байдинги. Вы лучше контроллер покажите и модель.
По скрину видно, что параметры в form, значит если их нет в модели, попробуйте указать атрибут FromForm. Атрибуты нужны, когда метод самостоятельно не понимает от куда параметры брать для парсинга.
И другой вариант, руками из реквеста их забрать. Но обычно этим способо пользуются когда надо проваледировать параметр до парсинга.

[DailyDDose]

eRKa, Посмотрите, пожалуйста, полностью исходный код проекта. (WebApiServer source)
Я уже просто не знаю что это за магия, у всех работает -- а у меня нет))

ValuesController

namespace WebApiServer.Controllers
{
    using System.Collections.Generic;
    using Microsoft.AspNetCore.Mvc;
    using Models;

    [Route("api/[controller]")]
    public class ValuesController : Controller
    {
        // GET api/values
        [HttpGet]
        public IEnumerable<string> Get()
        {
            return new string[] {"value1", "value2", "value3"};
        }

        // GET api/values/5
        [HttpGet("{id}")]
        public string Get(int id)
        {
            return "value";
        }

        // POST api/values
        [HttpPost]
        public string Post(/*[FromForm]*/ AuthModel Form)
        {
            return $"result: '{Form?.Login}__{Form?.Password}'";
        }

        // PUT api/values/5
        [HttpPut("{id}")]
        public void Put(int id, [FromBody] string value)
        {
        }

        // DELETE api/values/5
        [HttpDelete("{id}")]
        public void Delete(int id)
        {
        }
    }
}

AuthModel

using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;

namespace WebApiServer.Models
{
    public class AuthModel
    {
        public string Login { get; }
        public string Password { get; }
    }
}

[DailyDDose]

eRKa, впрочем, действительно, магия какая-то.
Оно заработало:

(()=>{
	const authData = {
		Login: 'admin',
		Password: 'rnf23nfr0wenf'
    }
	const options = {
		method: 'POST',
		headers: {
			Accept: 'application/json, text/plain, */*',
			'Content-Type': 'application/json'
        },
		body: JSON.stringify(authData)
	};
    fetch('http://localhost:53187/api/values', options)
        .then(response => response.text())
        .then(data => console.warn(data));
})();
undefined
// result: 'admin_rnf23nfr0wenf'

[eRKa]

DailyDDose, ты же моделям сетера не дал))

{
	public string Login { get; }
	public string Password { get; }
}

Это значит, что он может только читать из модели значения и не может присвоить. Нужно так

{
	public string Login { get; set; }
	public string Password { get; set; }
}

[eRKa]

В твоем проекте, если добавить сетеры в модель и в контроллере [FromBody]AuthModel, то все работает.

[DailyDDose]

eRKa, блин, опять моя невнимательность. Дело было лишь в аксессорах. Т.к. поля были ридонли вот они и не заполнялись)) Спасибо огромное, не лень же было разбираться))