Jobs::ProcessPost unable to process <img a="data:image


(PJH) #1

Admin required to access most of the URL’s here.

First, I’m not sure if this is even a ‘problem’ as such…

Brief Description: http://what.thedailywtf.com/t/sidekiq-investigation/1581

It seems <img a="data:...... is causing problems for Jobs::ProcessPost.

URL above attempts to de-construct a short post that has had repeated failures with each post in the format.

From: post concerned

`backtick enclosed` copy of what is being tested

raw copy of the stuff above

---
Generated:

<ProcessPost error, where applicable.>

Two of the three (first, second) caused more errors. <img a="data: appears to be what they have in common.


(Jeff Atwood) #2

Is this related to image processing / downloading @zogstrip? maybe those can be ignored.


(Kane York) #3

It’s choking on any <img> without a src.

Minimal example:

<img>
Twenty characters to make the post

(Régis Hanol) #4

Thanks @PJH & @riking, I just fixed both issues :rabbit:

https://github.com/discourse/discourse/commit/c7330ed73f7a967eb87ea24cc21c530c5fe5aa30


(Jeff Atwood) #5