{"action":"created","issue":{"url":"https://api.github.com/repos/aspnet/Mvc/issues/7602","repository_url":"https://api.github.com/repos/aspnet/Mvc","labels_url":"https://api.github.com/repos/aspnet/Mvc/issues/7602/labels{/name}","comments_url":"https://api.github.com/repos/aspnet/Mvc/issues/7602/comments","events_url":"https://api.github.com/repos/aspnet/Mvc/issues/7602/events","html_url":"https://github.com/aspnet/Mvc/issues/7602","id":311707292,"number":7602,"title":"Info-level logging about page execution is confusing","user":{"login":"dougbu","id":6431421,"avatar_url":"https://avatars1.githubusercontent.com/u/6431421?v=4","gravatar_id":"","url":"https://api.github.com/users/dougbu","html_url":"https://github.com/dougbu","followers_url":"https://api.github.com/users/dougbu/followers","following_url":"https://api.github.com/users/dougbu/following{/other_user}","gists_url":"https://api.github.com/users/dougbu/gists{/gist_id}","starred_url":"https://api.github.com/users/dougbu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dougbu/subscriptions","organizations_url":"https://api.github.com/users/dougbu/orgs","repos_url":"https://api.github.com/users/dougbu/repos","events_url":"https://api.github.com/users/dougbu/events{/privacy}","received_events_url":"https://api.github.com/users/dougbu/received_events","type":"User","site_admin":false},"labels":[{"id":93054382,"url":"https://api.github.com/repos/aspnet/Mvc/labels/1%20-%20Ready","name":"1 - Ready","color":"CCCCCC","default":false},{"id":70264660,"url":"https://api.github.com/repos/aspnet/Mvc/labels/bug","name":"bug","color":"fc2929","default":true}],"state":"open","locked":false,"assignee":{"login":"jbagga","id":6234561,"avatar_url":"https://avatars0.githubusercontent.com/u/6234561?v=4","gravatar_id":"","url":"https://api.github.com/users/jbagga","html_url":"https://github.com/jbagga","followers_url":"https://api.github.com/users/jbagga/followers","following_url":"https://api.github.com/users/jbagga/following{/other_user}","gists_url":"https://api.github.com/users/jbagga/gists{/gist_id}","starred_url":"https://api.github.com/users/jbagga/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbagga/subscriptions","organizations_url":"https://api.github.com/users/jbagga/orgs","repos_url":"https://api.github.com/users/jbagga/repos","events_url":"https://api.github.com/users/jbagga/events{/privacy}","received_events_url":"https://api.github.com/users/jbagga/received_events","type":"User","site_admin":false},"assignees":[{"login":"jbagga","id":6234561,"avatar_url":"https://avatars0.githubusercontent.com/u/6234561?v=4","gravatar_id":"","url":"https://api.github.com/users/jbagga","html_url":"https://github.com/jbagga","followers_url":"https://api.github.com/users/jbagga/followers","following_url":"https://api.github.com/users/jbagga/following{/other_user}","gists_url":"https://api.github.com/users/jbagga/gists{/gist_id}","starred_url":"https://api.github.com/users/jbagga/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbagga/subscriptions","organizations_url":"https://api.github.com/users/jbagga/orgs","repos_url":"https://api.github.com/users/jbagga/repos","events_url":"https://api.github.com/users/jbagga/events{/privacy}","received_events_url":"https://api.github.com/users/jbagga/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/aspnet/Mvc/milestones/42","html_url":"https://github.com/aspnet/Mvc/milestone/42","labels_url":"https://api.github.com/repos/aspnet/Mvc/milestones/42/labels","id":3008391,"number":42,"title":"2.2.0","description":null,"creator":{"login":"mkArtakMSFT","id":34246760,"avatar_url":"https://avatars3.githubusercontent.com/u/34246760?v=4","gravatar_id":"","url":"https://api.github.com/users/mkArtakMSFT","html_url":"https://github.com/mkArtakMSFT","followers_url":"https://api.github.com/users/mkArtakMSFT/followers","following_url":"https://api.github.com/users/mkArtakMSFT/following{/other_user}","gists_url":"https://api.github.com/users/mkArtakMSFT/gists{/gist_id}","starred_url":"https://api.github.com/users/mkArtakMSFT/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mkArtakMSFT/subscriptions","organizations_url":"https://api.github.com/users/mkArtakMSFT/orgs","repos_url":"https://api.github.com/users/mkArtakMSFT/repos","events_url":"https://api.github.com/users/mkArtakMSFT/events{/privacy}","received_events_url":"https://api.github.com/users/mkArtakMSFT/received_events","type":"User","site_admin":false},"open_issues":100,"closed_issues":1,"state":"open","created_at":"2018-01-02T18:28:59Z","updated_at":"2018-04-26T17:48:31Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2018-04-05T17:47:19Z","updated_at":"2018-05-03T20:50:18Z","closed_at":null,"author_association":"MEMBER","body":"Messages mix page and handler mentions with action mentions when logging about the same request e.g.\r\n```\r\nRoute matched with {page = \"/Account/Register\", area = \"Identity\", action = \"\", controller = \"\"}. Executing action /Account/Register\r\n...\r\nExecuted action /Account/Register in 2275.7977ms\r\n```\r\nversus (in between the two messages above)\r\n```\r\nExecuting handler method OnPostAsync with arguments () - ModelState is Valid\r\n```\r\n\r\nShould stick with consistent terms, preferably without expanding the meaning of \"action\". In addition, the `Executing handler method` message should fully qualify the method name, as we do in similar (true) action messages e.g.\r\n```\r\nExecuting action method netcoreIndividualAuth2_1Preview2.Controllers.HomeController.Index (netcoreIndividualAuth2_1Preview2) - Validation state: Valid\r\n```\r\n\r\n----\r\nFound while running through aspnet/Release#263"},"comment":{"url":"https://api.github.com/repos/aspnet/Mvc/issues/comments/386432359","html_url":"https://github.com/aspnet/Mvc/issues/7602#issuecomment-386432359","issue_url":"https://api.github.com/repos/aspnet/Mvc/issues/7602","id":386432359,"user":{"login":"dougbu","id":6431421,"avatar_url":"https://avatars1.githubusercontent.com/u/6431421?v=4","gravatar_id":"","url":"https://api.github.com/users/dougbu","html_url":"https://github.com/dougbu","followers_url":"https://api.github.com/users/dougbu/followers","following_url":"https://api.github.com/users/dougbu/following{/other_user}","gists_url":"https://api.github.com/users/dougbu/gists{/gist_id}","starred_url":"https://api.github.com/users/dougbu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dougbu/subscriptions","organizations_url":"https://api.github.com/users/dougbu/orgs","repos_url":"https://api.github.com/users/dougbu/repos","events_url":"https://api.github.com/users/dougbu/events{/privacy}","received_events_url":"https://api.github.com/users/dougbu/received_events","type":"User","site_admin":false},"created_at":"2018-05-03T20:50:18Z","updated_at":"2018-05-03T20:50:18Z","author_association":"MEMBER","body":"@jbagga looks much better than what we have today.\r\n\r\nI suggest we need to log about the model state even when executing an implicit handler. For example, is the `PageActionInvoker[101]`  logging dropped when executing a page with no `OnGet[Async]` method while processing a GET request?"}}
{"actor":{"display_login":"dougbu"}}