ASP.NET Core 2.1 RedirectToAction将TempData附加到url

Question

This is my save function. It is under a controller called Repositories. When I save a new Repository, it appends this to my url:

https://localhost:44325/Repositories/Repositories?TempData=%5BSuccess,%20Repository%20Created%5D&ViewBag=Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.DynamicViewData&HttpContext=Microsoft.AspNetCore.Http.DefaultHttpContext&Request=Microsoft.AspNetCore.Http.Internal.DefaultHttpRequest&Response=Microsoft.AspNetCore.Http.Internal.DefaultHttpResponse&RouteData=Microsoft.AspNetCore.Routing.RouteData&ModelState=%5BName,%20Microsoft.AspNetCore.Mvc.ModelBinding.ModelStateDictionary%2BModelStateNode%5D&ControllerContext=Microsoft.AspNetCore.Mvc.ControllerContext&MetadataProvider=Microsoft.AspNetCore.Mvc.ModelBinding.Metadata.DefaultModelMetadataProvider&ModelBinderFactory=Microsoft.AspNetCore.Mvc.ModelBinding.ModelBinderFactory&Url=Microsoft.AspNetCore.Mvc.Routing.UrlHelper&ObjectValidator=Microsoft.AspNetCore.Mvc.Internal.DefaultObjectValidator&User=System.Security.Claims.ClaimsPrincipal

I understand that it has something to do with RedirectToAction("Index",this) as when I use Redirect("Index"), the url just ends with a /Index. However, I would like the Index word to be hidden and RedirectToAction does that. How can I solve this issue?

public IActionResult Index()
{
    return View();
}

[HttpPost]
[ValidateAntiForgeryToken]
public ActionResult Save(CreateRepositoryViewModel Input)
{
    if (!ModelState.IsValid)
    {
        return View("Create", Input);
    }
    var directory = directoriesController.Create(Input);
    if (directory != null)
    {
        Input.Path = directory;
        var result = repositoriesData.Save(Input);
    }
    else
    {
        TempData["Error"] = "Repository Creation" + LoggingGlobals.Error;
        return RedirectToAction("Index", this);
    }
    TempData["Success"] = "Repository " + LoggingGlobals.Create;
    return RedirectToAction("Index", this);
}

Answer 1

The problem lies in this return statement:

return RedirectToAction("Index", this);

According to RedirectToAction overloads list , the second parameter may contain routeValues while an object passed to it instead of string:

public virtual RedirectToActionResult RedirectToAction (string actionName, object routeValues)

Hence, you're actually passing ControllerBase instance as routeValues parameter. Therefore, you should provide controller name instead:

return RedirectToAction("Index", "Repositories");

If you want to pass routeValues parameter together with controller name, use RedirectToAction with 3 overloads like this:

return RedirectToAction("Index", "Repositories", new { parameterName = "value" });

Note:

The RedirectToAction uses HTTP GET method, which passed route parameters as query string, therefore a viewmodel object doesn't fit into that. You should use another TempData or Session state instance to pass the viewmodel object to another controller.

TempData["ViewModel"] = Input;
return RedirectToAction("Index", "Repositories");

Answer 2

The main problem is already addressed in another answer.

I will address

However, I would like the Index word to be hidden

With attribute routing, you can set an empty route template

[Route("[controller]")]
public class RepositoriesController {


    [HttpGet]
    [Route("")] //GET Repositories
    [Route("[action]")] //GET Repositories/Index
    public IActionResult Index() {
        return View();
    }

    //...
}

That way when invoking return RedirectToAction("Index") , the generated URL will be what was configured in the route template.

Reference Routing to controller actions in ASP.NET Core