puzzle / decidim-zuerich

Mitwirkungsportal für die Stadt Zürich, basierend auf Decidim
https://mitwirken.stadt-zuerich.ch
GNU Affero General Public License v3.0
8 stars 2 forks source link

Newsletter: Logo wird nicht geladen #245

Closed larsUE closed 9 months ago

larsUE commented 2 years ago

Auf beiden Integrationsumgebungen werden in den Newslettern die Logos und Bilder derzeit nicht geladen, auch nicht im verschickten Mail:

Bildschirmfoto 2022-05-05 um 10 31 57

Case auf Mitwirken, keine Logos und Bilder geladen

Nachforschungen Wenn ich auf «Auf der Website anzeigen» im Mail klicke erscheint folgendes (es ist ein 404-Errror):

Bildschirmfoto 2022-05-05 um 10 34 02

Wenn ich das iFrame der Newsletter-Vorschau analysiere, finde ich folgende Fehlermeldung:

Bildschirmfoto 2022-05-05 um 10 35 05

Lustigerweise, wenn ich aber den angezeigten Link selbst klicke, lande ich bei folgender Fehlermeldung:

Bildschirmfoto 2022-05-05 um 10 36 44
Kagemaru commented 2 years ago

Hmm... Bei mir bekomme ich folgendes Mail: image

Und auch die Preview funktioniert bei mir: image

Den 404 bekomme ich nur, wenn ich den Preview Link anklicke bei einem noch nicht versendeten Newsletter (da dieser noch nicht öffentlich zugänglich ist).

@larsUE Kannst du das bestätigen? Ansonsten brauche ich einen Newsletter, bei dem ich dies nachvollziehen kann.

Kagemaru commented 2 years ago

Die URLs die ausgeliefert werden für die Bilder scheinen abzulaufen. Das müssen wir bei uns anpassen, damit permanente URLs verwendet werden.

larsUE commented 2 years ago

In dem Fall ist das ein Fehler der neuen Datenhaltung/ S3? Wenn der Newsletterversand reibungslos funktioniert (wie es derzeit scheint), dann können wir das gerne auch erst nach dem Update beheben.

Kagemaru commented 2 years ago

Mit S3 hat das wenig zu tun. Es wäre der Rails/Decidim gewesen, die falsche Links herausgeben. Aber ich kann hier entwarnen, der Link im Newsletter ist ok so. Er macht einen Redirect über Rails auf das Bild auf S3. Das ist das erwartete Verhalten.

Kannst du mal testen, ob alles so funktioniert, wie es soll?

larsUE commented 2 years ago

Ok, danke für die Erklärung. Ich erhalte den Newsletter auch wie gewünscht mit den Bildern. Somit ist nur die Vorschau defekt. -> Das können wir von mir aus auch erst nach dem Update flicken. Ich habe die Tags angepasst und die Ticket zurück ins ready verschoben.

larsUE commented 2 years ago

@Kagemaru und ich haben hier getestet und scheinbar ist dies ein Safari-spezifisches Caching-Problem. Der Newsletter Versand funktioniert ohne Probleme. Deshalb werden wir diesen Bug nicht flicken.