asp.net core Api - insert new row in database

Question

I'm trying to create an address object in a database through an asp.net core api. I'm using Postman to invoke the method.

Class:

namespace LC.Tools.API.Controllers {
    [Route("api/[controller]")]
    public class MailerController : Controller
    {
        Data.LCToolsDbContext _context;

        public MailerController(Data.LCToolsDbContext context)
        {
            _context = context;
        }

        [HttpPost]
        public async Task<IActionResult> CreateAddress(int client, string name, string email)
        {
            Address adr = new Address() { ClientId = client, Name = name, Email = email };
            _context.MailerAddresses.Add(adr);
            await _context.SaveChangesAsync();

            return Ok(adr);
        }
    } }

URL (using POST):

http://localhost:50444/api/mailer/createAddress?client=1&name=test&[email protected]

I also have a break point in the method, but it never hits. I don't get any error message, it just doesn't do anything.

Have a look at URL Encoding your values. An @ sign is not valid in a URL. Use %40, you'll be fine. — rickvdbosch
– rickvdbosch, Commented May 30, 2017 at 7:41

arslanaybars · Accepted Answer · 2017-05-30 08:11:35Z

3

You can see @Rick van den Bosch's comment but still you would like to specify the route with action. use this

 [Route("api/[controller]/[action]")]

answered May 30, 2017 at 8:11

arslanaybars

1,8533 gold badges24 silver badges29 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

rickvdbosch · Accepted Answer · 2017-05-30 08:08:38Z

1

See @Ayvaras' comment. Since you're building a Web API controller, you don't have to specify the action name in your URL. The HttpPost points all POST actions to the mailer controller towards the CreateAddress method.

Your POST URL should be:

http://localhost:50444/api/mailer?client=1&name=test&[email protected]

answered May 30, 2017 at 8:08

rickvdbosch

15.8k2 gold badges47 silver badges60 bronze badges

Comments

SteinTech · Accepted Answer · 2017-05-30 08:14:26Z

1

Problem solved! Thanks Ayvaras,

[Route("api/[controller]/[action]")]

did the trick. Now that the method is found I can look into how to pass an object instead of using querystring

answered May 30, 2017 at 8:14

SteinTech

4,1444 gold badges44 silver badges91 bronze badges

Comments

jorgonor · Accepted Answer · 2017-05-30 08:09:33Z

0

You are not using POST correctly.

If you use POST you should not be sending parameters through the query string, you should use the request body. Even though, if you still have to send them via query string you should use the FromQuery attribute in your action parameters.

[HttpPost]
public async Task<IActionResult> CreateAddress([FromQuery] int client, [FromQuery] string name, [FromQuery] string email)
{
    Address adr = new Address() { ClientId = client, Name = name, Email = email };
    _context.MailerAddresses.Add(adr);
    await _context.SaveChangesAsync();

    return Ok(adr);
}

Edit: Use [Route("api/[controller]/[action]")] to append the action name in your action route.

edited May 30, 2017 at 8:09

answered May 30, 2017 at 7:55

jorgonor

1,72911 silver badges16 bronze badges

2 Comments

poke Over a year ago

POST is not required to use the HTTP body only to communicate data. Just like you use the rest of the URL, you can just use query arguments at any moment, for any HTTP method. That being said, according to the comments, the endpoint is not found, so your suggestion will not help here.

jorgonor Over a year ago

@poke It's not required but it's not a convention. If a customer or developer has to implement this API it won't look like the usual thing to use POST but send parameters via query string.

JonasMH · Accepted Answer · 2017-05-30 09:40:07Z

0

Instead of changing the controllers routing scheme, it's also possible to specify the endpoint name by doing the following

[HttpPost("createAddress")]
public async Task<IActionResult> CreateAddress(int client, string name, string email)
{
    [...]
}

answered May 30, 2017 at 9:40

JonasMH

1,18311 silver badges21 bronze badges

Collectives™ on Stack Overflow

asp.net core Api - insert new row in database

5 Answers 5

Comments

Comments

Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

Comments

Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related