Multiple error messages for over-sized images


(loopback0 - TDWTF) #1

Repro steps (confirmed on here):

  • Copy large image (eg this) - copy the image itself, not a link.
  • Paste into message
  • Get error about size (or 2 errors sometimes)
  • Cancel composer window
  • Reopen it again
  • Paste large image
  • Get 1 more error than last time

This carries on increasing if you do everything in a single tab, gets worse if you try multiple tabs.


(Michael Downey) #2

Reproduced, but to be clear, the image itself must be copied, i.e., from your local computer, rather than copying & pasting the URL.


(loopback0 - TDWTF) #3

Yes, indeed. Clarified.

And thanks for spotting the missing category.


(Jeff Atwood) #4

Something for you to look at @sam and @zogstrip?


(Kane York) #5

Feels related to the upload changes recently made for S3, etc to work well.


(Sam Saffron) #6

Awesome report, will have a look shortly


(Sam Saffron) #7

Someone had a pretty bad typo there

https://github.com/discourse/discourse/commit/77c7bad61f54e8be199a3e2af5e6a7a9c0455c1d


(Sam Saffron) #8