S3 CDN URL ignored when uploading into posts

[DanielMarquard]

All of our traffic goes through CloudFront, and static content is fetched by CloudFront from one of several S3 buckets.

With uploads being sent to S3, Discourse embeds the content with a generic bucket URL despite our having s3 cdn url set. Once submitted, posts appear to use the correct URL set in s3 cdn url, although this may also be because we have DISCOURSE_CDN_URL set as well (both are set to the same URL).

The post preview window references the standard S3 URL seen in the unbaked post, though. This is a problem because we use a private ACL on files uploaded to our Discourse uploads bucket, as content on this bucket should never be accessed outside of CloudFront. The result is a broken image in the preview window.

[timkelty]

Enttäuschend, dass dies seit vielen Jahren gemeldet wird, ohne dass es eine Lösung gibt.

@DanielMarquard Haben Sie einen Workaround gefunden oder waren Sie gezwungen, Ihre S3-ACL öffentlich zu stellen?

[Stephen]

Ich bin mir nicht sicher, ob es fair ist zu behaupten, dass dies defekt ist, da die komplexere Implementierung nur unvollständig unterstützt wird.

S3-Uploads funktionieren wie dokumentiert und verhalten sich korrekt.

[timkelty]

@Stephen Auf jeden Fall interpretationsabhängig…

Für mich sieht es „kaputt

[riking (Kane York)]

Es könnte sich lohnen, dies jetzt zu beheben – AWS nimmt die Benachrichtigung über öffentliche Lese-ACLs immer ernster.

[sam (Sam Saffron)]

Da unsere aktuelle Implementierung kein IMG-Tag mehr enthält, z. B.:

![image|690x298,50%](upload://eGAOR4Xy4aRM2lwPIuTokBf60jS.png)

sollte es recht einfach sein, oben genannten Inhalt über ein CDN auszuliefern, was zudem den Datenverkehr reduziert.

@vinothkannans, könntest du das bitte so anpassen, dass die Vorschau die CDN-URL verwendet?

Anstatt dass die Vorschau Folgendes anzeigt:

<img src="//assets-meta-cdck-prod-meta.s3.dualstack.us-west-1.amazonaws.com/original/3X/6/6/66ef078b3ef9da9aa1b3356cb21fe22a0d25eaf0.png" alt="image" class="resizable" width="345" height="149">

sollte die CDN-URL angezeigt werden, und zwar diese:

<img src="https://d11a6trkgmumsb.cloudfront.net/original/3X/6/6/66ef078b3ef9da9aa1b3356cb21fe22a0d25eaf0.png" alt="image" class="resizable" width="345" height="149">

[vinothkannans (Vinoth Kannan)]

Jetzt ist es wie im folgenden Commit behoben

FIX: use upload's cdn url in composer preview if available.

[RGJ (Richard - Communiteq)]

Fortsetzung der Diskussion aus S3-CDN-URL wird beim Hochladen in Beiträge ignoriert:

Es scheint, als ob dieses Problem wieder aufgetreten ist. Ich kann es hier auf Meta reproduzieren.

[codinghorror (Jeff Atwood)]

Können wir das überprüfen, @vinothkannans oder @falco?

[vinothkannans (Vinoth Kannan)]

Gemäß der jüngsten Änderung erhalten wir die URL der hochgeladenen Datei nun direkt aus der ursprünglichen Upload-Anfrage. Zuvor haben wir sie über den Endpunkt /uploads/lookup-urls.json bezogen. Dieser Commit sollte das Problem beheben.

FIX: return cdn url for uploads if available.

[Falco]

Es sieht so aus, als würde diese Korrektur keine benutzerdefinierten Emojis in der Vorschau abdecken, zum Beispiel .

[applebee1558 (Alex Liu)]

Auch fehlen Anhänge wie Datenexporte oder beliebige Dateianhänge. Diese stammen weiterhin aus S3.

[nbianca (Bianca)]

Das kann ich beheben!

FIX: Use CDN for custom emojis (#11401)