{"action":"created","issue":{"url":"https://api.github.com/repos/stolendata/totp/issues/2","labels_url":"https://api.github.com/repos/stolendata/totp/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/stolendata/totp/issues/2/comments","events_url":"https://api.github.com/repos/stolendata/totp/issues/2/events","html_url":"https://github.com/stolendata/totp/pull/2","id":59856130,"number":2,"title":"optional parameters for getURI and static functions","user":{"login":"benaryorg","id":6145260,"avatar_url":"https://avatars.githubusercontent.com/u/6145260?v=3","gravatar_id":"","url":"https://api.github.com/users/benaryorg","html_url":"https://github.com/benaryorg","followers_url":"https://api.github.com/users/benaryorg/followers","following_url":"https://api.github.com/users/benaryorg/following{/other_user}","gists_url":"https://api.github.com/users/benaryorg/gists{/gist_id}","starred_url":"https://api.github.com/users/benaryorg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benaryorg/subscriptions","organizations_url":"https://api.github.com/users/benaryorg/orgs","repos_url":"https://api.github.com/users/benaryorg/repos","events_url":"https://api.github.com/users/benaryorg/events{/privacy}","received_events_url":"https://api.github.com/users/benaryorg/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-04T20:23:01Z","updated_at":"2015-03-05T11:09:32Z","closed_at":"2015-03-05T11:09:32Z","pull_request":{"url":"https://api.github.com/repos/stolendata/totp/pulls/2","html_url":"https://github.com/stolendata/totp/pull/2","diff_url":"https://github.com/stolendata/totp/pull/2.diff","patch_url":"https://github.com/stolendata/totp/pull/2.patch"},"body":"already tested"},"comment":{"url":"https://api.github.com/repos/stolendata/totp/issues/comments/77346269","html_url":"https://github.com/stolendata/totp/pull/2#issuecomment-77346269","issue_url":"https://api.github.com/repos/stolendata/totp/issues/2","id":77346269,"user":{"login":"stolendata","id":1296914,"avatar_url":"https://avatars.githubusercontent.com/u/1296914?v=3","gravatar_id":"","url":"https://api.github.com/users/stolendata","html_url":"https://github.com/stolendata","followers_url":"https://api.github.com/users/stolendata/followers","following_url":"https://api.github.com/users/stolendata/following{/other_user}","gists_url":"https://api.github.com/users/stolendata/gists{/gist_id}","starred_url":"https://api.github.com/users/stolendata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stolendata/subscriptions","organizations_url":"https://api.github.com/users/stolendata/orgs","repos_url":"https://api.github.com/users/stolendata/repos","events_url":"https://api.github.com/users/stolendata/events{/privacy}","received_events_url":"https://api.github.com/users/stolendata/received_events","type":"User","site_admin":false},"created_at":"2015-03-05T11:09:32Z","updated_at":"2015-03-05T11:09:32Z","body":"But what exactly does SailOTP do when you supply digits/period to it? Ignore them? Behave oddly? If it just ignores them then there is no problem - a larger QR code can at most be considered an inconvenience.\r\n\r\nRFC6238 recommends 30 seconds as the default period but offers no recommendation on the digit count. Since 6 digits is presumed to be the default by everyone (Google Authenticator even hardcodes this and ignores the period parameter) I'll go for your suggestion and make the two parameters optional along with some simple format checking."}}