Richie
2018 年12 月 16 日 13:15
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
2018 年12 月 16 日 13:18
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
2018 年12 月 16 日 13:27
3
Although here’s an older email that does display the image:
So I’m not sure now…
Richie
2018 年12 月 16 日 13:33
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?
It’s a known bug of the digest logo. Looks nearly identical in cause though, the malformed URLs when hosted in S3.
2 个赞
Per the linked topic, this bug was fixed though, so patch to latest if you haven’t already.
2 个赞
Richie
2018 年12 月 17 日 07:48
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
2018 年12 月 18 日 16:30
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
2018 年12 月 18 日 16:32
9
Another observation, images in the daily summary emails do actually appear
Are emails with single notifications + an image handled differently somehow?
Richie
2019 年1 月 9 日 18:32
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
2019 年2 月 20 日 13:08
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
2019 年3 月 3 日 19:36
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
2019 年3 月 3 日 20:31
13
Does your site have force_https enabled?
1 个赞
sam
2019 年3 月 3 日 20:36
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 个赞
Stephen
2019 年3 月 3 日 20:38
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 个赞
Richie
2019 年3 月 3 日 21:37
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
2019 年3 月 3 日 21:54
17
Are you using a CDN? If so what is your CDN URL?
Richie
2019 年3 月 4 日 07:15
18
Hi @sam there is no CDN in use on my instance:
2 个赞
Richie
2019 年3 月 6 日 08:48
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
2019 年3 月 7 日 01:11
20
hmmm maybe the codepath the omits images from ultra short emails is accidentally being triggered for digests, is that possible @neil ?
2 个赞
