{"action":"created","issue":{"url":"https://api.github.com/repos/msimerson/mail-dmarc/issues/58","labels_url":"https://api.github.com/repos/msimerson/mail-dmarc/issues/58/labels{/name}","comments_url":"https://api.github.com/repos/msimerson/mail-dmarc/issues/58/comments","events_url":"https://api.github.com/repos/msimerson/mail-dmarc/issues/58/events","html_url":"https://github.com/msimerson/mail-dmarc/pull/58","id":60597934,"number":58,"title":"Time align reports","user":{"login":"marcbradshaw","id":6148082,"avatar_url":"https://avatars.githubusercontent.com/u/6148082?v=3","gravatar_id":"","url":"https://api.github.com/users/marcbradshaw","html_url":"https://github.com/marcbradshaw","followers_url":"https://api.github.com/users/marcbradshaw/followers","following_url":"https://api.github.com/users/marcbradshaw/following{/other_user}","gists_url":"https://api.github.com/users/marcbradshaw/gists{/gist_id}","starred_url":"https://api.github.com/users/marcbradshaw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marcbradshaw/subscriptions","organizations_url":"https://api.github.com/users/marcbradshaw/orgs","repos_url":"https://api.github.com/users/marcbradshaw/repos","events_url":"https://api.github.com/users/marcbradshaw/events{/privacy}","received_events_url":"https://api.github.com/users/marcbradshaw/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/msimerson/mail-dmarc/labels/Needs%20Tests","name":"Needs Tests","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-03-11T01:44:00Z","updated_at":"2015-03-11T02:13:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/msimerson/mail-dmarc/pulls/58","html_url":"https://github.com/msimerson/mail-dmarc/pull/58","diff_url":"https://github.com/msimerson/mail-dmarc/pull/58.diff","patch_url":"https://github.com/msimerson/mail-dmarc/pull/58.patch"},"body":"Fixes Issue #55.\r\n\r\nAlign report times to a UTC day based window where possible.\r\n\r\nWould appreciate comments before merge."},"comment":{"url":"https://api.github.com/repos/msimerson/mail-dmarc/issues/comments/78187818","html_url":"https://github.com/msimerson/mail-dmarc/pull/58#issuecomment-78187818","issue_url":"https://api.github.com/repos/msimerson/mail-dmarc/issues/58","id":78187818,"user":{"login":"msimerson","id":261635,"avatar_url":"https://avatars.githubusercontent.com/u/261635?v=3","gravatar_id":"","url":"https://api.github.com/users/msimerson","html_url":"https://github.com/msimerson","followers_url":"https://api.github.com/users/msimerson/followers","following_url":"https://api.github.com/users/msimerson/following{/other_user}","gists_url":"https://api.github.com/users/msimerson/gists{/gist_id}","starred_url":"https://api.github.com/users/msimerson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/msimerson/subscriptions","organizations_url":"https://api.github.com/users/msimerson/orgs","repos_url":"https://api.github.com/users/msimerson/repos","events_url":"https://api.github.com/users/msimerson/events{/privacy}","received_events_url":"https://api.github.com/users/msimerson/received_events","type":"User","site_admin":false},"created_at":"2015-03-11T02:13:41Z","updated_at":"2015-03-11T02:13:41Z","body":"Looks great Marc, nice work. I'd like to see just two additions:\r\n* test coverage for the new functions\r\n* config settings for the min / max intervals. Something like this:\r\n\r\n```ini\r\n--- a/share/mail-dmarc.ini\r\n+++ b/share/mail-dmarc.ini\r\n@@ -19,10 +19,17 @@ pass    =\r\n ; enables 'save by default'\r\n auto_save = 0\r\n \r\n+[report_sending]\r\n+; minimum reporting interval in seconds: default: none\r\n+; min_interval = 3600\r\n+;\r\n+; maximum reporting interval in seconds: default: none\r\n+; max_interval = 86400\r\n+\r\n```"}}