{"action":"created","issue":{"url":"https://api.github.com/repos/grafov/m3u8/issues/40","repository_url":"https://api.github.com/repos/grafov/m3u8","labels_url":"https://api.github.com/repos/grafov/m3u8/issues/40/labels{/name}","comments_url":"https://api.github.com/repos/grafov/m3u8/issues/40/comments","events_url":"https://api.github.com/repos/grafov/m3u8/issues/40/events","html_url":"https://github.com/grafov/m3u8/pull/40","id":139598745,"number":40,"title":"Parsing SCTE35 tag from playlist","user":{"login":"vishal24tuniki","id":11678056,"avatar_url":"https://avatars.githubusercontent.com/u/11678056?v=3","gravatar_id":"","url":"https://api.github.com/users/vishal24tuniki","html_url":"https://github.com/vishal24tuniki","followers_url":"https://api.github.com/users/vishal24tuniki/followers","following_url":"https://api.github.com/users/vishal24tuniki/following{/other_user}","gists_url":"https://api.github.com/users/vishal24tuniki/gists{/gist_id}","starred_url":"https://api.github.com/users/vishal24tuniki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vishal24tuniki/subscriptions","organizations_url":"https://api.github.com/users/vishal24tuniki/orgs","repos_url":"https://api.github.com/users/vishal24tuniki/repos","events_url":"https://api.github.com/users/vishal24tuniki/events{/privacy}","received_events_url":"https://api.github.com/users/vishal24tuniki/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/grafov/m3u8/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-03-09T14:54:17Z","updated_at":"2016-03-15T19:45:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/grafov/m3u8/pulls/40","html_url":"https://github.com/grafov/m3u8/pull/40","diff_url":"https://github.com/grafov/m3u8/pull/40.diff","patch_url":"https://github.com/grafov/m3u8/pull/40.patch"},"body":"Adding feature as per SCTE standards\r\nRef : http://www.scte.org/documents/pdf/standards/SCTE%2067%202014.pdf **12.1.6.1 HLS Cue Tags**"},"comment":{"url":"https://api.github.com/repos/grafov/m3u8/issues/comments/196991889","html_url":"https://github.com/grafov/m3u8/pull/40#issuecomment-196991889","issue_url":"https://api.github.com/repos/grafov/m3u8/issues/40","id":196991889,"user":{"login":"bradleyfalzon","id":1834577,"avatar_url":"https://avatars.githubusercontent.com/u/1834577?v=3","gravatar_id":"","url":"https://api.github.com/users/bradleyfalzon","html_url":"https://github.com/bradleyfalzon","followers_url":"https://api.github.com/users/bradleyfalzon/followers","following_url":"https://api.github.com/users/bradleyfalzon/following{/other_user}","gists_url":"https://api.github.com/users/bradleyfalzon/gists{/gist_id}","starred_url":"https://api.github.com/users/bradleyfalzon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bradleyfalzon/subscriptions","organizations_url":"https://api.github.com/users/bradleyfalzon/orgs","repos_url":"https://api.github.com/users/bradleyfalzon/repos","events_url":"https://api.github.com/users/bradleyfalzon/events{/privacy}","received_events_url":"https://api.github.com/users/bradleyfalzon/received_events","type":"User","site_admin":false},"created_at":"2016-03-15T19:45:51Z","updated_at":"2016-03-15T19:45:51Z","body":"I'm also thinking that these SCTE tags should be added to it's own struct, such as:\r\n\r\n```\r\ntype MediaSegment struct {\r\n\tSeqId           uint64\r\n\tTitle           string\r\n\tURI             string\r\n\tDuration        float64\r\n\tLimit           int64\r\n\tOffset          int64\r\n\tKey             *Key\r\n\tMap             *Map\r\n\tSCTE\t\t\t*SCTE\r\n\tDiscontinuity   bool\r\n\tProgramDateTime time.Time\r\n}\r\n\r\ntype SCTE struct {\r\n\tCue string\r\n\tID string\r\n\tTime time.Time\r\n}\r\n```\r\n\r\nThis way it's easier to add extra tags in the future, and contains all SCTE in the same way it's done for Key and Map. This is a slightly more substantial change to the code, but I think it's straight forward to do and makes sense in relation to how other optional tags are handled.\r\n\r\nThoughts?"}}