IUrlHelper and support for RouteOptions.LowercaseUrls

[bugproof]

I have the following setting:

services.Configure<RouteOptions>(opt => opt.LowercaseUrls = true);

When I generate URL from my controller, query parameters are named like in C#, for example

query?Property=x

I expected it to be query?property=x, ideally in camel case. So query?propertySomething=x

It would be nice if IUrlHelper implementation or RouteValueDictionary or something else respected this setting.

Example invocation:

Url.Action(nameof(MyAction), myModel);

//myModel
class MyModel
{
    public int PropertySomething { get; set; }
}

[mkArtakMSFT]

Thanks for contacting us, @enemyofthedawn.
We've moved this issue to the Backlog milestone. This means that it is not going to happen for the coming release. We will reassess the backlog following the current release and consider this item at that time. However, keep in mind that there are many other high priority features with which it will be competing for resources.

[mkArtakMSFT]

@enemyofthedawn, if you'd like to submit a PR in the Routing repo to handle this with a separate option, we'd be something we'd consider to take.

[bugproof]

Also it would be nice if IUrlHelper could generate hyphenated query strings, and some possibility to choose convention to use when generating links (camelcase etc.) that means some ability to support:

• query?PropertySomething=x
• query?propertySomething=x
• query?property-something=x

It could be a separate option to use hyphens (boolean) or enum option to choose the convention (something like Json.NET does)

Or maybe some action so you can write your function that controls the naming.

What do you think?

[rynowak]

Fixed by acea87a

[rynowak]

@LeCoque - if you'd be willing to send a PR, I think we'd be willing to take something like this

Or maybe some action so you can write your function that controls the naming.

I don't want to add too many options to the framework by default - as you point out there are a lot of different schemes for generating query strings. I'd rather add a hook that lets application developers customize it.