Opened 2 years ago
Closed 2 years ago
#17495 closed defect (bug) (fixed)
blip.tv oEmbeds don't work
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.2 |
| Component: | Media | Version: | 3.2 |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
It seems that by some reason, blip.tv oEmbeds don't work. Probably related to the fact they use a different URL for videos right now. If I test it using http://blip.tv/oembed with current URL's still retrieves the correct code, but still don't work in WordPress.
Attachments (2)
Change History (8)
- Keywords has-patch added
nacin was correct about removing /file/ but a coworker and I also found that there was an error in the JSON returned by blip.tv.
I'm not sure if you want to introduce a workaround into core since it's their problem but I'm attaching two diffs. One resolves just the WordPress problem. The other includes a workaround the blip.tv's error.
What precisely is the blip.tv formatting error? I've emailed with them before, and they got back to me (with a fix upstream) within like 24 hours.
It looks like they're missing a comma after the author_url value:
http://blip.tv/oembed/?url=http://blip.tv/often-awesome-the-series/often-awesome-the-series-episode-1-diagnosis-3038972
I spoke with one of their employees on Twitter but an email would probably be good:
http://twitter.com/#!/glennansley/statuses/71225170060247040
Jake Simms from blip says it should be repaired on their end shortly so 17495-wp-fix.diff should be sufficient.
Looks like 'http://blip.tv/file/*' needs to become 'http://blip.tv/* based on the new structure there.
I think they also used to leverage subdomains for shows but that seems to be gone.