Refactor AccessDenied methods and clean up usings (Log entries create…

…d from user input)

src/Web/Grand.Web.Admin/Controllers/HomeController.cs

@@ -197,17 +197,16 @@ public async Task<IActionResult> GetStatesByCountryId([FromServices] ICountrySer
         return Json(result);
     }
 
-    public async Task<IActionResult> AccessDenied(string pageUrl)
+    public async Task<IActionResult> AccessDenied()
     {
         var currentCustomer = _workContext.CurrentCustomer;
         if (currentCustomer == null || await _groupService.IsGuest(currentCustomer))
         {
-            _logger.LogInformation("Access denied to anonymous request on {PageUrl}", pageUrl);
+            _logger.LogInformation("Access denied to anonymous request");
             return View();
         }
 
-        _logger.LogInformation("Access denied to user #{CurrentCustomerEmail} \'{CurrentCustomerEmail}\' on {PageUrl}",
-            currentCustomer.Email, currentCustomer.Email, pageUrl);
+        _logger.LogInformation("Access denied to user #{CurrentCustomerEmail} \'{CurrentCustomerEmail}\'", currentCustomer.Email, currentCustomer.Email);
 
         return View();
     }

src/Web/Grand.Web.Common/Controllers/BaseController.cs

@@ -4,10 +4,8 @@
 using Grand.Web.Common.Events;
 using Grand.Web.Common.Extensions;
 using Grand.Web.Common.Filters;
-using Grand.Web.Common.Models;
 using Grand.Web.Common.Page;
 using MediatR;
-using Microsoft.AspNetCore.Http.Extensions;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.AspNetCore.Mvc.Filters;
 using Microsoft.AspNetCore.Mvc.ModelBinding;
@@ -216,7 +214,7 @@ protected virtual async Task AddLocales<TLocalizedModelLocal>(ILanguageService l
     /// <returns>Access denied view</returns>
     protected IActionResult AccessDeniedView()
     {
-        return RedirectToAction("AccessDenied", "Home", new { pageUrl = HttpContext.Request.GetEncodedPathAndQuery() });
+        return RedirectToAction("AccessDenied", "Home");
     }
 
     /// <summary>

src/Web/Grand.Web.Vendor/Controllers/HomeController.cs

@@ -44,10 +44,9 @@ public IActionResult Statistics()
         return View();
     }
 
-    public IActionResult AccessDenied(string pageUrl)
+    public IActionResult AccessDenied()
     {
-        _logger.LogInformation("Access denied to user #{CurrentCustomerEmail} on {PageUrl}",
-            _workContext.CurrentCustomer.Email, pageUrl);
+        _logger.LogInformation("Access denied to user #{CurrentCustomerEmail}", _workContext.CurrentCustomer.Email);
         return View();
     }