{"action":"created","issue":{"url":"https://api.github.com/repos/concrete5/concrete5-legacy/issues/1922","labels_url":"https://api.github.com/repos/concrete5/concrete5-legacy/issues/1922/labels{/name}","comments_url":"https://api.github.com/repos/concrete5/concrete5-legacy/issues/1922/comments","events_url":"https://api.github.com/repos/concrete5/concrete5-legacy/issues/1922/events","html_url":"https://github.com/concrete5/concrete5-legacy/pull/1922","id":108174232,"number":1922,"title":"Adding time to getString to add to the entropy and decrease collision…","user":{"login":"ddproxy","id":105473,"avatar_url":"https://avatars.githubusercontent.com/u/105473?v=3","gravatar_id":"","url":"https://api.github.com/users/ddproxy","html_url":"https://github.com/ddproxy","followers_url":"https://api.github.com/users/ddproxy/followers","following_url":"https://api.github.com/users/ddproxy/following{/other_user}","gists_url":"https://api.github.com/users/ddproxy/gists{/gist_id}","starred_url":"https://api.github.com/users/ddproxy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ddproxy/subscriptions","organizations_url":"https://api.github.com/users/ddproxy/orgs","repos_url":"https://api.github.com/users/ddproxy/repos","events_url":"https://api.github.com/users/ddproxy/events{/privacy}","received_events_url":"https://api.github.com/users/ddproxy/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-09-24T17:16:42Z","updated_at":"2015-09-25T02:28:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/concrete5/concrete5-legacy/pulls/1922","html_url":"https://github.com/concrete5/concrete5-legacy/pull/1922","diff_url":"https://github.com/concrete5/concrete5-legacy/pull/1922.diff","patch_url":"https://github.com/concrete5/concrete5-legacy/pull/1922.patch"},"body":"… opportunity\r\n\r\nThis is not perfect, but it mitigates the problem a little."},"comment":{"url":"https://api.github.com/repos/concrete5/concrete5-legacy/issues/comments/143106666","html_url":"https://github.com/concrete5/concrete5-legacy/pull/1922#issuecomment-143106666","issue_url":"https://api.github.com/repos/concrete5/concrete5-legacy/issues/1922","id":143106666,"user":{"login":"KorvinSzanto","id":1007419,"avatar_url":"https://avatars.githubusercontent.com/u/1007419?v=3","gravatar_id":"","url":"https://api.github.com/users/KorvinSzanto","html_url":"https://github.com/KorvinSzanto","followers_url":"https://api.github.com/users/KorvinSzanto/followers","following_url":"https://api.github.com/users/KorvinSzanto/following{/other_user}","gists_url":"https://api.github.com/users/KorvinSzanto/gists{/gist_id}","starred_url":"https://api.github.com/users/KorvinSzanto/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KorvinSzanto/subscriptions","organizations_url":"https://api.github.com/users/KorvinSzanto/orgs","repos_url":"https://api.github.com/users/KorvinSzanto/repos","events_url":"https://api.github.com/users/KorvinSzanto/events{/privacy}","received_events_url":"https://api.github.com/users/KorvinSzanto/received_events","type":"User","site_admin":false},"created_at":"2015-09-25T02:28:43Z","updated_at":"2015-09-25T02:28:43Z","body":"Then I think hashing in the user id is a good fix since we already make\nthese tokens unique per user.\nOn Thu, Sep 24, 2015 at 7:15 PM Jon West <notifications@github.com> wrote:\n\n> As per the uuid suggestion - a uuid nor random hash should be used as a\n> validator or authentication method. Even if the hash is bcrypt or md5.\n> Using uuid or rebuilding this unique hash method serves to ensure a valid\n> unique identifier for data and in my opinion should not be used for\n> sensitive identification in the manner of the original issue I filed.\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/concrete5/concrete5-legacy/pull/1922#issuecomment-143104387>\n> .\n>\n"}}