When the image is re-written from plain text, it only alters the display. However, when using a WYSIWYG editor, it'll also alter the display on input as well.

It'd be nice to put the external image in a data tag that can be manually grabbed by the end user so they can fetch the image and make it appear locally.

CommentFileSizeAuthor
#2 2714775-external_location_store-1.patch1.37 KBjaperry

Comments

japerry created an issue. See original summary.

japerry’s picture

japerry
Location KOMK
commented
Status: Active » Needs review
StatusFileSize
new 2714775-external_location_store-1.patch1.37 KB
drumm’s picture

drumm
he/him
Location NY, US
commented

I'm not sure why the branch wasn't passing. I ran fresh tests on the branch, and it is passing. I went ahead and forward ported this, #2715627: Keep a copy of original img src in _filter_html_image_secure_process().

  • drumm committed d996980 on 7.x-1.x authored by japerry 
    Issue #2714775 by japerry: Store location of original image
drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Needs review » Fixed

Works as advertised, committed.

  • drumm committed 6213201 on 7.x-1.x 
    Issue #2714775: Add testing

  • drumm committed 45ec798 on 7.x-1.x 
    Issue #2714775: Re-add rewriting
hass’s picture

hass commented

Are there plans to cut a new release soon?

drumm’s picture

drumm
he/him
Location NY, US
commented

I may do another beta release. No full release until #1737022: Host Whitelist has tests and everything.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.