If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

SoundCloud API appears to be reporting invalid duration

RESOLVED FIXED

Status

Webmaker
Popcorn Maker
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: mjschranz, Assigned: mjschranz)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
This is causing tracks to be incorrectly reported in their duration and have clips much shorter than they actually are.

For example:

https://soundcloud.com/a-jameshetfield/metallica-nothing-else-matters // Reports 1:49
http://soundcloud.com/vithaya/interview // Reports 1:00

Comment 1

4 years ago
in the two cases being reported, the metadata returned does not match the clip being requested.  Instead, the metadata belongs to an older clip belonging to another user whose title (terminal element of URL) is identical to the requested clip's:  

the url requested was: 
https://soundcloud.com/macthesavage/interview  

info returned was for:

http://soundcloud.com/vithaya/interview

etc.
(Assignee)

Comment 2

4 years ago
Nope, bug on our end. It's simply because we aren't using their API optimally.

Fix inc.
(Assignee)

Comment 3

4 years ago
Created attachment 8364499 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/427
Attachment #8364499 - Flags: review?(scott)
Comment on attachment 8364499 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/427

Weird that the other way doesn't work, but not really worth fussing over.
Attachment #8364499 - Flags: review?(scott) → review+

Comment 5

4 years ago
Commit pushed to master at https://github.com/mozilla/popcorn.webmaker.org

https://github.com/mozilla/popcorn.webmaker.org/commit/b3bbfefed4af45767421aa5b8f76d8ba098bc772
Fix Bug 962650 - Fix issue where SoundCloud clips had incorrect data due to multiple clips with same title

Updated

4 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.