[Bug]: FromRoute parameter only binds directly on method signature, not when nested in request record about swashbuckle.aspnetcore

[RasmusEdvardsen]

On further examination, this actually works if the [FromRoute] decorator within the request record has a Name attribute specified, like so:

[FromRoute(Name = "id")]

I was under the impression that this would be inferred somehow - or is this indeed the intended behavior?

[jgarciadelanoceda]

@RasmusEdvardsen. The issue that you are describing here should be fixed just by adding [FromRoute] on the object property.
But if you do that, you have to specify the Route in the controller as Id instead of id( It's case sensitive).

Do you want it to be case insensitive?

[RasmusEdvardsen]

@jgarciadelanoceda Ah I see, that might be what I'm referring to in my latest comment. If you ask me my personal preference, I'd just want it to work the same way it does as if it was specified directly on the controller parameter. If it already does, I apologize for taking time out of your schedule. Thanks for a good project, btw.

[jgarciadelanoceda]

I'll look at this feature, because it seems to be a issue of the library although with dotnet 8 you will have the issue
dotnet/aspnetcore#54212. It's a nice to have case insensitive on the Route parameters

[RasmusEdvardsen]

Agreed - and yea, it does seem like the issue you linked pertains to this issue. I went around it the dirty way - suppressing the warning.

[jgarciadelanoceda]

As far as I have seen it's an issue with the JS that is part of the /swagger-api/swagger-ui package, I was not able to get a way to make the filtering by Path case insensitive with this library :(

[jgarciadelanoceda]

@RasmusEdvardsen. The issue that you are describing here should be fixed just by adding [FromRoute] on the object property.
But if you do that, you have to specify the Route as Id instead of id( It's case sensitive).

I think that when this issue is solved, we could handle swagger-api/swagger-ui#8143

[RasmusEdvardsen]

Alright, sounds good, thanks. I'll close this, and let's hope the linked issue will be fixed soon 👍

[RasmusEdvardsen]

@jgarciadelanoceda do you want me to close this for now? I realised I should've asked you first, as you have linked this one in swagger-api/swagger-ui#8143

[jgarciadelanoceda]

As you please, I showed that it has linked a PR that has been active for 3 years, but I hope they merge it

[RasmusEdvardsen]

Okay, I'll close it. Thanks