Richie
December 16, 2018, 1:15pm
1
Hi everyone,
Not sure if this is a bug or a configuration issue. However, I’m quite sure this has only happened recently, I’m sure I’d have noticed before now otherwise
All our images are hosted on AWS.
We are running Discourse v2.2.2.beta6+5
When an email notification arrives that contains an embedded image on Amazon AWS, the image itself is missing / not displaying.
This issue may be of a similar theme / issue to Missing logo in email summary ? (Which we also suffered with, and was subsequently fixed, thanks!)
Here’s a screen shot of how the email(s) look when viewed in the Mail app on iOS devices:
The image links to //greyarrows.s3.dualstack.eu-west-2.amazonaws.com/original/2X/3/37430494c198530c000587d93a8fd05ebfdd97f3.jpeg
If I put https: in front of the link and open it in a new private browser window, the image does seem to load ok: https://greyarrows.s3.dualstack.eu-west-2.amazonaws.com/original/2X/3/37430494c198530c000587d93a8fd05ebfdd97f3.jpeg
Is this a bug? Or have I possibly missed some kind of config setting on AWS and simply not noticed this issue before?
Richie
December 16, 2018, 1:18pm
2
Although, when viewed in Gmail on Firefox on my desktop PC, the image appears perfectly:
Which makes me think this may just be an iOS thing…?
Richie
December 16, 2018, 1:27pm
3
Although here’s an older email that does display the image:
So I’m not sure now…
Richie
December 16, 2018, 1:33pm
4
Seemed to work in emails I received in October, but sadly I don’t subscribe to many threads for email notifications.
Anyone else experienced this issue?
chumstick
December 17, 2018, 3:21am
5
It’s a known bug of the digest logo. Looks nearly identical in cause though, the malformed URLs when hosted in S3.
2 Likes
Per the linked topic, this bug was fixed though, so patch to latest if you haven’t already.
2 Likes
Richie
December 17, 2018, 7:48am
7
Hi @chumstick and @codinghorror
As per my first post, I’m already running Discourse v2.2.2.beta6+5 which did indeed fix the digest logo bug
@chumstick is the same problem that caused the logo issue recently also affecting S3 images in the body of the email too?
Richie
December 18, 2018, 4:30pm
8
Here’s one from yesterday that also doesnt display the image:
The link that’s presented doesn’t include https: at the start, will this be an issue?
Richie
December 18, 2018, 4:32pm
9
Another observation, images in the daily summary emails do actually appear
Are emails with single notifications + an image handled differently somehow?
Richie
January 9, 2019, 6:32pm
10
Just wondering if anybody has any suggestions on a fix or work around for images in posts not appearing within the email notifications?
To confirm, it only occurs when using the built in / default Mail app on iOS. This issue presents itself on iOS 10 and iOS 12 (I don’t have an 11 device).
Richie
February 20, 2019, 1:08pm
11
All of our notification emails that reference images uploaded to our AWS still fail to display when viewed using the Mail app built in to iOS.
The daily summary emails display the images fine - which I don’t understand. They must be embedded or referenced differently somehow.
Here’s what all iOS users see:
As a test, I’d like to add https: to the start of the image src url as I think this may solve the problem:
Does anyone know how I could do that?
Would adding https: to the img src url cause issues elsewhere?
Richie
March 3, 2019, 7:36pm
12
I think my theory of the https: missing from the image URL may be correct.
I received an email notification from meta today, and it included an image:
As you can see, that image displayed perfectly in the iOS mail app.
Looking at the URL of the image itself, it DOES include https: at the start:
I realise that the image in that email is hosted on fastly rather than AWS, but why does meta put https in the image URL and my instance does not?
Stephen
March 3, 2019, 8:31pm
13
Does your site have force_https enabled?
1 Like
sam
March 3, 2019, 8:36pm
14
I wonder @codinghorror if we should default force https in production now, it pops up enough, and non https sites should be a rarity
3 Likes
Stephen
March 3, 2019, 8:38pm
15
There are lots of situations where Let’s Encrypt can fail, but if enrolment succeeds in a standard install I would be all for it.
We can detect if 443 is exposed, the SSL/LE templates are in place and the certbot result, right?
1 Like
Richie
March 3, 2019, 9:37pm
16
Hi @Stephen - yes, it’s been on from day one:
Curious as to why meta puts https in the image url of the email notifications and my discourse (v2.3.0.beta3) does not?
sam
March 3, 2019, 9:54pm
17
Are you using a CDN? If so what is your CDN URL?
Richie
March 4, 2019, 7:15am
18
Hi @sam there is no CDN in use on my instance:
2 Likes
Richie
March 6, 2019, 8:48am
19
I’m seeing this issue in the daily summary emails sometimes too:
And again, when looking at the URL of the image there is no https: in there either:
I still don’t know if this is the cause and until I can find a way to add https: to the image url I don’t know how else to rule this out.
sam
March 7, 2019, 1:11am
20
hmmm maybe the codepath the omits images from ultra short emails is accidentally being triggered for digests, is that possible @neil ?
2 Likes
