Routing problems by not providing default route values

While working on a new site that uses Phil Haack’s RouteMagic I came across a curious thing. All my Html.ActionLink calls were routing to a controller/action that they shouldn’t route to. I fired up Glimpse to see what routes were possible matches to try and find the culprit. (The route exists in an external library) Here’s an example:

@Html.ActionLink("Log on", "LogOn", "Account")

This would normally route to /account/logon however if you have a routes.MapDelegate call that has no {parameters} it will match every time.

Given the following routes

routes.MapDelegate("RouteFailureTest", "home/test", context => new HomeController().HomeTest(context.HttpContext));

    "Default", // Route name
    "{controller}/{action}/{id}", // URL with parameters
    new { controller = "Home", action = "Index", id = UrlParameter.Optional }//, // Parameter defaults

a call to @Html.ActionLink(“Log Off”, “Logout”, “User”) always matches the “RouteFailureTest”.

After checking the source code from Codeplex I can see that no default route values are specified in the call (here is the source from RouteMagic)

public static Route MapHttpHandler(this RouteCollection routes, IHttpHandler httpHandler, string name, string url, object defaults, object constraints) {
    var route = new Route(url, new HttpHandlerRouteHandler(httpHandler));
    route.Defaults = new RouteValueDictionary(defaults); //I believe this is where the problem lies
    route.Constraints = new RouteValueDictionary(constraints);
    routes.Add(name, new NormalizeRoute(route));
    return route;

Since Html.ActionLink and it’s brethren supply their own Controller/Action values my thoughts were that the lack of values in the route caused it to match. To confirm this is the case I created another standard route with null as it’s default parameter values.

    "ShouldntMatch", // Route name
    "home/test", // URL with parameters
    null // Parameter defaults

This caused the same issue as RouteMagic’s MapDelegate.

Normally all routes have default values but MapDelegate tells the router exactly where it should go to handle a particular route. This means that it bypasses the DefaultControllerFactory which utilizes the route values to determine where the url should route to. This would normally be fine except for the way that ActionLink determines which route to format the url after.

There really doesn’t seem to be much that we can do to get around this except to provide a default to the MapDelegate route that will never hit.

var route = routes.MapDelegate("RouteFailureTest", "home/test",
    context => new HomeController().HomeTest(context.HttpContext));

route.Defaults = new RouteValueDictionary(new { controller = "thisisacontrollerthatwillneverhit", action = "thisisanactionthatwillneverhit" });

This is such an interesting case. It relies on the fact that the route has no default controller, action or parameters such as {id}. Yet it breaks a lot of other MVC functions that rely on routes having those values. For now I’ve hard coded the particular issue I have with this and replace other Html.ActionLinks with Html.RouteLink but this is only a temporary solution until either Phil Haack can resolve it or I can find another solution. For now I’m modifying the specific route that is causing problems with the unhittable controller/action. It seems to work but I definitely can’t rely on it.

- Ben Dornis

blog comments powered by Disqus