Problème de chemin relatif provoquant une erreur CORS sur les sites Discourse

Yt.w · Octobre 27, 2024, 6:27

Le code suivant dans lib/highlight_js.rb fonctionne sur certains domaines mais pas sur d’autres :

def self.path
    "/highlight-js/#{Discourse.current_hostname}/#{version SiteSetting.highlighted_languages}.js"
end

J’ai deux sites Discourse. Ce code fonctionne sur le domaine www.abc.com, mais il ne fonctionne pas sur efg.com. Dans la console du navigateur, je vois l’erreur suivante :

formatter.js:383 Uncaught (in promise) TypeError: Failed to resolve module specifier '/highlight-js/efg.com/9797975efac87d28baa695ae13ca72ccaf5120f5.js'. The base URL is about:blank because import() is called from a CORS-cross-origin script.

Après avoir modifié self.path dans highlight_js.rb comme suit :

def self.path
    "https://#{Discourse.current_hostname}/highlight-js/#{Discourse.current_hostname}/#{version SiteSetting.highlighted_languages}.js"
end

le problème a été résolu.

Il semble que l’utilisation d’un chemin relatif comme
import('/highlight-js/efg.com/9797975efac87d28baa695ae13ca72ccaf5120f5.js')
provoque un problème CORS, tandis que l’utilisation d’une URL absolue,
import('https://efg.com/highlight-js/efg.com/9797975efac87d28baa695ae13ca72ccaf5120f5.js') , fonctionne bien.

NateDhaliwal · Octobre 27, 2024, 8:22

Étrange que cela fonctionne avec abc.com mais pas avec efg.com, puisque les 2 différentes routes sont complètement différentes

Yt.w · Octobre 27, 2024, 9:04

Je veux dire que cela fonctionne avec www.abc.com mais pas avec efg.com, je ne suis pas sûr pourquoi, mais cela se produit dans mes cas

simonk · Octobre 28, 2024, 11:49

Utilisez-vous un CDN sur l’un de ces sites ? J’ai remarqué que la coloration du code ne fonctionne plus sur mon site, et je pense que c’est à cause de ceci :

Dans mon cas, mon CDN ne renvoie pas d’en-tête Access-Control-Allow-Origin pour le fichier highlightjs. Je remarque que le CDN de Meta inclut cet en-tête, je me demande donc ce qui est différent.

$ curl --silent -I https://d3bpeqsaub0i6y.cloudfront.net/highlight-js/meta.discourse.org/9797975efac87d28baa695ae13ca72ccaf5120f5.js | grep -i access-control
access-control-allow-origin: *
access-control-allow-methods: GET, HEAD, OPTIONS

Cependant, ces en-têtes ne sont pas servis par le serveur d’origine :

$ curl --silent -I https://meta.discourse.org/highlight-js/meta.discourse.org/9797975efac87d28baa695ae13ca72ccaf5120f5.js | grep -i access-control

D’après ce que je peux voir, Discourse est censé ajouter des en-têtes access-control aux fichiers highlightjs :

github.com/discourse/discourse

app/controllers/highlight_js_controller.rb

27c20eeac


      
          # frozen_string_literal: true
          
          class HighlightJsController < ApplicationController
            skip_before_action :preload_json,
                               :redirect_to_login_if_required,
                               :redirect_to_profile_if_required,
                               :check_xhr,
                               :verify_authenticity_token,
                               only: [:show]
          
            before_action :apply_cdn_headers, only: [:show]
          
            def show
              no_cookies
          
              RailsMultisite::ConnectionManagement.with_hostname(params[:hostname]) do
                current_version = HighlightJs.version(SiteSetting.highlighted_languages)
          
                return redirect_to path(HighlightJs.path) if current_version != params[:version]
          
                # note, this can be slightly optimised by caching the bundled file, it cuts down on N reads

Cependant, ces en-têtes ne sont appliqués que si la requête est une « requête CDN » :

github.com/discourse/discourse

app/controllers/application_controller.rb

27c20eeac


      
            return unless mini_profiler_enabled?
            Rack::MiniProfiler.authorize_request
          end
          
          def check_xhr
            # bypass xhr check on PUT / POST / DELETE provided api key is there, otherwise calling api is annoying
            return if !request.get? && (is_api? || is_user_api?)
            raise ApplicationController::RenderEmpty.new if !request.format&.json? && !request.xhr?
          end
          
          def apply_cdn_headers
            if Discourse.is_cdn_request?(request.env, request.method)
              Discourse.apply_cdn_headers(response.headers)
            end
          end
          
          def self.requires_login(arg = {})
            @requires_login_arg = arg
          end
          
          def self.requires_login_arg

github.com/discourse/discourse

lib/discourse.rb

27c20eeac


      
          end
          
          mattr_accessor :redis
          
          def self.is_parallel_test?
            ENV["RAILS_ENV"] == "test" && ENV["TEST_ENV_NUMBER"]
          end
          
          CDN_REQUEST_METHODS ||= %w[GET HEAD OPTIONS]
          
          def self.is_cdn_request?(env, request_method)
            return if CDN_REQUEST_METHODS.exclude?(request_method)
          
            cdn_hostnames = GlobalSetting.cdn_hostnames
            return if cdn_hostnames.blank?
          
            requested_hostname = env[REQUESTED_HOSTNAME] || env[Rack::HTTP_HOST]
            cdn_hostnames.include?(requested_hostname)
          end
          
          def self.apply_cdn_headers(headers)

Cela ne fonctionne que si Discourse est configuré avec un nom d’hôte distinct pour les « requêtes CDN ».

simonk · Octobre 28, 2024, 12:02

Il existe un paramètre, cdn_origin_hostname, qui pourrait peut-être être défini sur le nom d’hôte normal de Discourse :

github.com/discourse/discourse

app/models/global_setting.rb

27c20eeac


      
          end
          
          # for testing
          def self.reset_s3_cache!
            @use_s3 = nil
          end
          
          def self.cdn_hostnames
            hostnames = []
            hostnames << URI.parse(cdn_url).host if cdn_url.present?
            hostnames << cdn_origin_hostname if cdn_origin_hostname.present?
            hostnames
          end
          
          def self.database_config
            hash = { "adapter" => "postgresql" }
          
            %w[
              pool
              connect_timeout
              socket

github.com/discourse/discourse

config/discourse_defaults.conf

27c20eeac


      
          
          # authorization key that will be included as a header in requests made by the
          # snapshots transporter to the URL specified above. The destination should
          # know this key and only accept requests that have this key in the
          # `Mini-Profiler-Transport-Auth` header.
          mini_profiler_snapshots_transport_auth_key =
          
          # recommended, cdn used to access assets
          cdn_url =
          
          # The hostname used by the CDN to request assets
          cdn_origin_hostname =
          
          # comma delimited list of emails that have developer level access
          developer_emails =
          
          # redis server address
          redis_host = localhost
          
          # redis server port
          redis_port = 6379

Je pense que cela ferait passer la vérification is_cdn_request, mais je ne sais pas s’il y aurait des effets secondaires négatifs.

Yt.w · Octobre 28, 2024, 12:48

Oui, j’utilise un CDN sur efg.com, mais lorsque j’accède au fichier js, il me renvoie un Access-Control-Allow-Origin correct, ce qui me laisse assez perplexe.

simonk · Octobre 29, 2024, 5:48

Pour faire suite, j’ai défini cdn_origin_hostname sur le nom de domaine normal de mon instance Discourse, j’ai vidé le cache du CDN, et maintenant highlightjs fonctionne à nouveau. Je n’ai pas remarqué d’effets secondaires…

cuo_wu · Mai 18, 2025, 11:34

Je rencontre aussi ce problème. Des mises à jour ?

Sujet		Réponses	Vues
Code highlighting failed because bunny.net CDN Support cdn , cors	4	1169	Mars 8, 2024
How to enable syntax highlighting? Support	5	97	Octobre 27, 2024
Aborting! CDN must have a protocol specified Support	7	939	Mai 27, 2019
Need relative URLs for uploaded images Support	8	1283	Janvier 22, 2021
Image upload, relative path instead of full path Support	6	1096	Juin 8, 2024

Problème de chemin relatif provoquant une erreur CORS sur les sites Discourse

Sujets connexes