Emoji Picker verwendet keine CDN-URLs für benutzerdefinierte Emojis

Fehler
1

Beschreibung

Standard-Emoji-Pfade werden nicht in CDN-URLs konvertiert, während Admin-Emoji-Pfade ordnungsgemäß konvertiert werden.

Beweise

Standard-Controller (keine CDN-Konvertierung):

Admin-Controller (korrekte Konvertierung mit EmojiSerializer):

Standard-Emoji-Pfade sollten wie Admin-Emoji-Pfade in CDN-URLs konvertiert werden.

2 „Gefällt mir“
2

Sehen Sie diese URLs irgendwo in der Benutzeroberfläche? Ich vermute, dass sie in der JS-App in CDN-URLs umgewandelt werden, bevor sie angezeigt werden.

3

Ich habe nachgesehen und kann bestätigen, dass die Emojis direkt S3-URLs (beginnend mit //) in der gerenderten Ausgabe verwenden, nicht CDN-URLs.

4

Auf welcher Seite sehen Sie das?

5

Sie können dies auf der Beitragsverfasserseite sehen, wenn Sie den Emoji-Auswahl verwenden. Wenn benutzerdefinierte Emojis geladen werden, verwenden sie direkt S3-Endpunkte, wie zum Beispiel:

https://assets-meta-cdck-prod-meta.s3.dualstack.us-west-1.amazonaws.com/original/3X/1/d/1de8f76096f76cf393640bdcff387ec9ae33dd3e.gif

Dies zeigt, dass die Pfade nicht wie erwartet in CDN-URLs umgewandelt werden.

6

image
image2408×416 228 KB

image
image2002×896 114 KB

7

Benutzerdefinierte Emoji testen :awthanks:

:up_arrow: Dies wird vom S3-CDN ausgeliefert… Wir haben ein Problem mit dem Picker selbst @j.jaffeux

Die Bildbeschreibung zeigt eine Suchergebnisseite mit verschiedenen Smiley-Emojis in einem Abschnitt namens „SMILEYS AND EMOTION“. (KI-generierte Bildunterschrift)
Die Bildbeschreibung zeigt eine Suchergebnisseite mit verschiedenen Smiley-Emojis in einem Abschnitt namens „SMILEYS AND EMOTION“. (KI-generierte Bildunterschrift)580×455 70.1 KB

Verwendet keine benutzerdefinierten Emojis vom S3-CDN

Diese Benutzeroberfläche verwendet das S3-CDN

Die Bildbeschreibung zeigt einen Screenshot eines Autovervollständigungs-Dropdown-Menüs mit dem Präfix „badger“, das mehrere vorgeschlagene Begriffe auflistet, darunter „badminton“, „badpokerface“, „supervillain“ und „-1“. (KI-generierte Bildunterschrift)
Die Bildbeschreibung zeigt einen Screenshot eines Autovervollständigungs-Dropdown-Menüs mit dem Präfix „badger“, das mehrere vorgeschlagene Begriffe auflistet, darunter „badminton“, „badpokerface“, „supervillain“ und „-1“. (KI-generierte Bildunterschrift)403×282 20.4 KB

Dies ist also sehr spezifisch für den Emoji-Picker, Verbesserung des Titels hier.

1 „Gefällt mir“
8

Danke, das sollte es beheben:

3 „Gefällt mir“
9

Vielen Dank für Ihren PR, der die CDN-Nutzung für benutzerdefinierte Emojis ermöglicht.

Während Ihre Frontend-Korrektur Emoji-CDN-Adressen korrekt anfordert, mache ich mir Sorgen um die Backend-Sicherheit.
Der Endpunkt https://meta.discourse.org/emojis.json gibt immer noch S3-Quell-URLs wie //assets-meta-cdck-prod-meta.s3.dualstack.us-west-1.amazonaws.com/original/3X/6/1/61e481320406f0f82ed780db3f04056128191613.png preis, was potenziell zu böswilligem Missbrauch und übermäßiger S3-Abrechnung führen könnte.

Für bessere Sicherheit schlage ich vor, direkt CDN-Adressen zurückzugeben, ähnlich wie bei nicht benutzerdefinierten Emojis.

Hier ist mein Versuch, den Code zu korrigieren:

class EmojisController < ApplicationController
  def index
    emojis = Emoji.allowed.group_by(&:group)
    
    emojis.each do |_, emoji_list|
      emoji_list.each do |emoji|
        emoji.url = Discourse.store.cdn_url(emoji.url) if emoji.url.present?
      end
    end
    
    render json: MultiJson.dump(emojis)
  end
  def search_aliases
    render json: MultiJson.dump(Emoji.search_aliases)
  end
end
10

Ja, ich weiß, dass wir es direkt für nicht benutzerdefinierte Emojis in emoji.rb machen, was ich dem Controller vorziehen würde. Ich möchte sicherstellen, dass dies keine Probleme verursacht, daher habe ich vorerst die einfachste Lösung gefunden, werde mich aber in Zukunft damit befassen.

2 „Gefällt mir“
11

Schließe, da das ursprüngliche Problem behoben ist. Du kannst gerne eine neue Feature-Anfrage für die Nachverfolgung eröffnen.

Diese Assets sind winzig und im Allgemeinen öffentlich, das Risiko erscheint gering.

2 „Gefällt mir“