Opened 3 years ago
Last modified 2 years ago
#14946 new enhancement
Only enforce OEmbed whitelisting for dangerous types
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Future Release |
| Component: | General | Version: | 3.0.1 |
| Severity: | normal | Keywords: | needs-patch dev-feedback |
| Cc: |
Description
Of the four OEmbed response types, only two (video and rich) are potentially dangerous. We should be able to allow all photo and link responses, without a whitelist.
Change History (3)
- Keywords needs-patch dev-feedback added
@ryan, @westi, @azaozz, can you weigh in here? If this is going to get in, needs a patch asap to beat freeze. @filosofo's counter-argument sounds reasonable, though.
Note: See
TracTickets for help on using
tickets.

I'm not sure that we can consider photo responses safe, in the sense that we use the value of the url parameter as the source for an image element.
In the past there have been security exploits (such as the GDI exploit) that used image files to trick clients into executing code.
Even the best-case scenario allows the remote server to set and read cookies.