{"action":"created","comment":{"url":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/comments/91217620","pull_request_review_id":11727150,"id":91217620,"diff_hunk":"@@ -239,10 +239,12 @@ public static void Verify(string payloadJson, string decodedCrypto, string decod\n         {\n             if (decodedCrypto != decodedSignature)\n             {\n-\t            var signatureVerificationException = new SignatureVerificationException(\"Invalid signature\");\n-\t\t\t\tsignatureVerificationException.Data.Add(\"Expected\", decodedCrypto);\n-\t\t\t\tsignatureVerificationException.Data.Add(\"Received\", decodedSignature);\n-\t            throw signatureVerificationException;\n+               var signatureVerificationException = new SignatureVerificationException(\"Invalid signature\")","path":"src/JWT/JWT.cs","position":null,"original_position":8,"commit_id":"9534dd5e64e29f030287b16a37e9b080aaf6ce91","original_commit_id":"0e843311aeb49a18539a6e03b91eebf3ffc82373","user":{"login":"abatishchev","id":351644,"avatar_url":"https://avatars.githubusercontent.com/u/351644?v=3","gravatar_id":"","url":"https://api.github.com/users/abatishchev","html_url":"https://github.com/abatishchev","followers_url":"https://api.github.com/users/abatishchev/followers","following_url":"https://api.github.com/users/abatishchev/following{/other_user}","gists_url":"https://api.github.com/users/abatishchev/gists{/gist_id}","starred_url":"https://api.github.com/users/abatishchev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abatishchev/subscriptions","organizations_url":"https://api.github.com/users/abatishchev/orgs","repos_url":"https://api.github.com/users/abatishchev/repos","events_url":"https://api.github.com/users/abatishchev/events{/privacy}","received_events_url":"https://api.github.com/users/abatishchev/received_events","type":"User","site_admin":false},"body":"I think that one good enough, especially if we'll reside the helper method here. Pelsse go ahead and rebase, I'll merge","created_at":"2016-12-07T02:45:42Z","updated_at":"2016-12-07T02:45:42Z","html_url":"https://github.com/jwt-dotnet/jwt/pull/64#discussion_r91217620","pull_request_url":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/64","_links":{"self":{"href":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/comments/91217620"},"html":{"href":"https://github.com/jwt-dotnet/jwt/pull/64#discussion_r91217620"},"pull_request":{"href":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/64"}}},"pull_request":{"url":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/64","id":96745343,"html_url":"https://github.com/jwt-dotnet/jwt/pull/64","diff_url":"https://github.com/jwt-dotnet/jwt/pull/64.diff","patch_url":"https://github.com/jwt-dotnet/jwt/pull/64.patch","issue_url":"https://api.github.com/repos/jwt-dotnet/jwt/issues/64","number":64,"state":"open","locked":false,"title":"TokenExpiredException inherits from SignatureVerificationException + Refactor ","user":{"login":"dstj","id":467692,"avatar_url":"https://avatars.githubusercontent.com/u/467692?v=3","gravatar_id":"","url":"https://api.github.com/users/dstj","html_url":"https://github.com/dstj","followers_url":"https://api.github.com/users/dstj/followers","following_url":"https://api.github.com/users/dstj/following{/other_user}","gists_url":"https://api.github.com/users/dstj/gists{/gist_id}","starred_url":"https://api.github.com/users/dstj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dstj/subscriptions","organizations_url":"https://api.github.com/users/dstj/orgs","repos_url":"https://api.github.com/users/dstj/repos","events_url":"https://api.github.com/users/dstj/events{/privacy}","received_events_url":"https://api.github.com/users/dstj/received_events","type":"User","site_admin":false},"body":"... and Refactor SignatureVerificationException to use properties instead of the Data collection directly","created_at":"2016-12-06T15:29:46Z","updated_at":"2016-12-07T02:45:42Z","closed_at":null,"merged_at":null,"merge_commit_sha":"db041bf16e63ac98888fc4979cdeb0cdd44d7465","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/64/commits","review_comments_url":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/64/comments","review_comment_url":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/comments{/number}","comments_url":"https://api.github.com/repos/jwt-dotnet/jwt/issues/64/comments","statuses_url":"https://api.github.com/repos/jwt-dotnet/jwt/statuses/9534dd5e64e29f030287b16a37e9b080aaf6ce91","head":{"label":"dstj:PR_SignationVerificationException_Improvement","ref":"PR_SignationVerificationException_Improvement","sha":"9534dd5e64e29f030287b16a37e9b080aaf6ce91","user":{"login":"dstj","id":467692,"avatar_url":"https://avatars.githubusercontent.com/u/467692?v=3","gravatar_id":"","url":"https://api.github.com/users/dstj","html_url":"https://github.com/dstj","followers_url":"https://api.github.com/users/dstj/followers","following_url":"https://api.github.com/users/dstj/following{/other_user}","gists_url":"https://api.github.com/users/dstj/gists{/gist_id}","starred_url":"https://api.github.com/users/dstj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dstj/subscriptions","organizations_url":"https://api.github.com/users/dstj/orgs","repos_url":"https://api.github.com/users/dstj/repos","events_url":"https://api.github.com/users/dstj/events{/privacy}","received_events_url":"https://api.github.com/users/dstj/received_events","type":"User","site_admin":false},"repo":{"id":44389084,"name":"jwt","full_name":"dstj/jwt","owner":{"login":"dstj","id":467692,"avatar_url":"https://avatars.githubusercontent.com/u/467692?v=3","gravatar_id":"","url":"https://api.github.com/users/dstj","html_url":"https://github.com/dstj","followers_url":"https://api.github.com/users/dstj/followers","following_url":"https://api.github.com/users/dstj/following{/other_user}","gists_url":"https://api.github.com/users/dstj/gists{/gist_id}","starred_url":"https://api.github.com/users/dstj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dstj/subscriptions","organizations_url":"https://api.github.com/users/dstj/orgs","repos_url":"https://api.github.com/users/dstj/repos","events_url":"https://api.github.com/users/dstj/events{/privacy}","received_events_url":"https://api.github.com/users/dstj/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/dstj/jwt","description":"JWT (JSON Web Token) implementation for .NET 3.5+ (Public Domain)","fork":true,"url":"https://api.github.com/repos/dstj/jwt","forks_url":"https://api.github.com/repos/dstj/jwt/forks","keys_url":"https://api.github.com/repos/dstj/jwt/keys{/key_id}","collaborators_url":"https://api.github.com/repos/dstj/jwt/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/dstj/jwt/teams","hooks_url":"https://api.github.com/repos/dstj/jwt/hooks","issue_events_url":"https://api.github.com/repos/dstj/jwt/issues/events{/number}","events_url":"https://api.github.com/repos/dstj/jwt/events","assignees_url":"https://api.github.com/repos/dstj/jwt/assignees{/user}","branches_url":"https://api.github.com/repos/dstj/jwt/branches{/branch}","tags_url":"https://api.github.com/repos/dstj/jwt/tags","blobs_url":"https://api.github.com/repos/dstj/jwt/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/dstj/jwt/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/dstj/jwt/git/refs{/sha}","trees_url":"https://api.github.com/repos/dstj/jwt/git/trees{/sha}","statuses_url":"https://api.github.com/repos/dstj/jwt/statuses/{sha}","languages_url":"https://api.github.com/repos/dstj/jwt/languages","stargazers_url":"https://api.github.com/repos/dstj/jwt/stargazers","contributors_url":"https://api.github.com/repos/dstj/jwt/contributors","subscribers_url":"https://api.github.com/repos/dstj/jwt/subscribers","subscription_url":"https://api.github.com/repos/dstj/jwt/subscription","commits_url":"https://api.github.com/repos/dstj/jwt/commits{/sha}","git_commits_url":"https://api.github.com/repos/dstj/jwt/git/commits{/sha}","comments_url":"https://api.github.com/repos/dstj/jwt/comments{/number}","issue_comment_url":"https://api.github.com/repos/dstj/jwt/issues/comments{/number}","contents_url":"https://api.github.com/repos/dstj/jwt/contents/{+path}","compare_url":"https://api.github.com/repos/dstj/jwt/compare/{base}...{head}","merges_url":"https://api.github.com/repos/dstj/jwt/merges","archive_url":"https://api.github.com/repos/dstj/jwt/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/dstj/jwt/downloads","issues_url":"https://api.github.com/repos/dstj/jwt/issues{/number}","pulls_url":"https://api.github.com/repos/dstj/jwt/pulls{/number}","milestones_url":"https://api.github.com/repos/dstj/jwt/milestones{/number}","notifications_url":"https://api.github.com/repos/dstj/jwt/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/dstj/jwt/labels{/name}","releases_url":"https://api.github.com/repos/dstj/jwt/releases{/id}","deployments_url":"https://api.github.com/repos/dstj/jwt/deployments","created_at":"2015-10-16T14:08:23Z","updated_at":"2016-12-05T22:51:17Z","pushed_at":"2016-12-06T19:26:48Z","git_url":"git://github.com/dstj/jwt.git","ssh_url":"git@github.com:dstj/jwt.git","clone_url":"https://github.com/dstj/jwt.git","svn_url":"https://github.com/dstj/jwt","homepage":"Install-Package JWT","size":5899,"stargazers_count":0,"watchers_count":0,"language":"C#","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"jwt-dotnet:master","ref":"master","sha":"e27176d24002b50ecf46c5cbf7185f9c16e4156d","user":{"login":"jwt-dotnet","id":12436799,"avatar_url":"https://avatars.githubusercontent.com/u/12436799?v=3","gravatar_id":"","url":"https://api.github.com/users/jwt-dotnet","html_url":"https://github.com/jwt-dotnet","followers_url":"https://api.github.com/users/jwt-dotnet/followers","following_url":"https://api.github.com/users/jwt-dotnet/following{/other_user}","gists_url":"https://api.github.com/users/jwt-dotnet/gists{/gist_id}","starred_url":"https://api.github.com/users/jwt-dotnet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jwt-dotnet/subscriptions","organizations_url":"https://api.github.com/users/jwt-dotnet/orgs","repos_url":"https://api.github.com/users/jwt-dotnet/repos","events_url":"https://api.github.com/users/jwt-dotnet/events{/privacy}","received_events_url":"https://api.github.com/users/jwt-dotnet/received_events","type":"Organization","site_admin":false},"repo":{"id":1921154,"name":"jwt","full_name":"jwt-dotnet/jwt","owner":{"login":"jwt-dotnet","id":12436799,"avatar_url":"https://avatars.githubusercontent.com/u/12436799?v=3","gravatar_id":"","url":"https://api.github.com/users/jwt-dotnet","html_url":"https://github.com/jwt-dotnet","followers_url":"https://api.github.com/users/jwt-dotnet/followers","following_url":"https://api.github.com/users/jwt-dotnet/following{/other_user}","gists_url":"https://api.github.com/users/jwt-dotnet/gists{/gist_id}","starred_url":"https://api.github.com/users/jwt-dotnet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jwt-dotnet/subscriptions","organizations_url":"https://api.github.com/users/jwt-dotnet/orgs","repos_url":"https://api.github.com/users/jwt-dotnet/repos","events_url":"https://api.github.com/users/jwt-dotnet/events{/privacy}","received_events_url":"https://api.github.com/users/jwt-dotnet/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/jwt-dotnet/jwt","description":"JWT (JSON Web Token) implementation for .NET 3.5+ (Public Domain)","fork":false,"url":"https://api.github.com/repos/jwt-dotnet/jwt","forks_url":"https://api.github.com/repos/jwt-dotnet/jwt/forks","keys_url":"https://api.github.com/repos/jwt-dotnet/jwt/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jwt-dotnet/jwt/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jwt-dotnet/jwt/teams","hooks_url":"https://api.github.com/repos/jwt-dotnet/jwt/hooks","issue_events_url":"https://api.github.com/repos/jwt-dotnet/jwt/issues/events{/number}","events_url":"https://api.github.com/repos/jwt-dotnet/jwt/events","assignees_url":"https://api.github.com/repos/jwt-dotnet/jwt/assignees{/user}","branches_url":"https://api.github.com/repos/jwt-dotnet/jwt/branches{/branch}","tags_url":"https://api.github.com/repos/jwt-dotnet/jwt/tags","blobs_url":"https://api.github.com/repos/jwt-dotnet/jwt/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jwt-dotnet/jwt/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jwt-dotnet/jwt/git/refs{/sha}","trees_url":"https://api.github.com/repos/jwt-dotnet/jwt/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jwt-dotnet/jwt/statuses/{sha}","languages_url":"https://api.github.com/repos/jwt-dotnet/jwt/languages","stargazers_url":"https://api.github.com/repos/jwt-dotnet/jwt/stargazers","contributors_url":"https://api.github.com/repos/jwt-dotnet/jwt/contributors","subscribers_url":"https://api.github.com/repos/jwt-dotnet/jwt/subscribers","subscription_url":"https://api.github.com/repos/jwt-dotnet/jwt/subscription","commits_url":"https://api.github.com/repos/jwt-dotnet/jwt/commits{/sha}","git_commits_url":"https://api.github.com/repos/jwt-dotnet/jwt/git/commits{/sha}","comments_url":"https://api.github.com/repos/jwt-dotnet/jwt/comments{/number}","issue_comment_url":"https://api.github.com/repos/jwt-dotnet/jwt/issues/comments{/number}","contents_url":"https://api.github.com/repos/jwt-dotnet/jwt/contents/{+path}","compare_url":"https://api.github.com/repos/jwt-dotnet/jwt/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jwt-dotnet/jwt/merges","archive_url":"https://api.github.com/repos/jwt-dotnet/jwt/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jwt-dotnet/jwt/downloads","issues_url":"https://api.github.com/repos/jwt-dotnet/jwt/issues{/number}","pulls_url":"https://api.github.com/repos/jwt-dotnet/jwt/pulls{/number}","milestones_url":"https://api.github.com/repos/jwt-dotnet/jwt/milestones{/number}","notifications_url":"https://api.github.com/repos/jwt-dotnet/jwt/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jwt-dotnet/jwt/labels{/name}","releases_url":"https://api.github.com/repos/jwt-dotnet/jwt/releases{/id}","deployments_url":"https://api.github.com/repos/jwt-dotnet/jwt/deployments","created_at":"2011-06-19T23:46:02Z","updated_at":"2016-12-06T07:22:12Z","pushed_at":"2016-12-07T02:44:53Z","git_url":"git://github.com/jwt-dotnet/jwt.git","ssh_url":"git@github.com:jwt-dotnet/jwt.git","clone_url":"https://github.com/jwt-dotnet/jwt.git","svn_url":"https://github.com/jwt-dotnet/jwt","homepage":"Install-Package JWT","size":6009,"stargazers_count":372,"watchers_count":372,"language":"C#","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":144,"mirror_url":null,"open_issues_count":11,"forks":144,"open_issues":11,"watchers":372,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/64"},"html":{"href":"https://github.com/jwt-dotnet/jwt/pull/64"},"issue":{"href":"https://api.github.com/repos/jwt-dotnet/jwt/issues/64"},"comments":{"href":"https://api.github.com/repos/jwt-dotnet/jwt/issues/64/comments"},"review_comments":{"href":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/64/comments"},"review_comment":{"href":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/jwt-dotnet/jwt/pulls/64/commits"},"statuses":{"href":"https://api.github.com/repos/jwt-dotnet/jwt/statuses/9534dd5e64e29f030287b16a37e9b080aaf6ce91"}}}}
{"actor":{"display_login":"abatishchev"}}