Percorso source map errato nell'installazione in sottocartella

My subfolder install kept giving errors in Firefox console about the fact that it is unable to find source maps.

Source map error: request failed with status 404 Resource URL: [https://redacted.com/community/assets/admin-…1d82fc1b0c7d1f24b7b71256a4eb4860f1aa96444925a361d8c24425c.js](https://redacted.com/community/assets/admin-e1c3cbc1d82fc1b0c7d1f24b7b71256a4eb4860f1aa96444925a361d8c24425c.js) Source Map URL: /assets/admin-e1c3cbc1d82fc1b0c7d1f24b7b71256a4eb4860f1aa96444925a361d8c24425c.js.map

As you can see, the /community path prefix is missing from the source map URL.

So I did a little digging.

lib/tasks/assets.rake does this:

  assets = cdn_relative_path("/assets")
  source_map_root = assets + ((d = File.dirname(from)) == "." ? "" : "/#{d}")

Now cdn_relative_path in lib/global_path.rb looks like this.

module GlobalPath
  def path(p)
    "#{GlobalSetting.relative_url_root}#{p}"
  end

  def cdn_path(p)
    GlobalSetting.cdn_url.blank? ? p : "#{GlobalSetting.cdn_url}#{path(p)}"
  end

  def upload_cdn_path(p)
    if SiteSetting.Upload.s3_cdn_url.present?
      p = Discourse.store.cdn_url(p)
    end

    (p =~ /^http/ || p =~ /^\/\//) ? p : cdn_path(p)
  end

  def cdn_relative_path(path)
    if (cdn_url = GlobalSetting.cdn_url).present?
      URI.parse(cdn_url).path + path
    else
      path
    end
  end

As you can see, both cdn_path and upload_cdn_path eventually call the path method which makes sure that the subfolder is being prefixed. However, cdn_relative_path does not call the path method, it just returns its parameter (called… path).

BTW I think cdn_path needs to return path(p) in the middle part of the ternary as well?

1 Mi Piace

@sam is the expert on source maps

@david has fussed with this in the past, but yeah getting the paths right for CDN + subfolder is not easy. I am open to a PR that fixes it provided this is very targeted at your magic combo and does not regress any places that work now.

3 Mi Piace

@sam Abbiamo alcune soluzioni per questo problema con una source map errata? Ho lo stesso problema con l’installazione in una sottocartella e Fastly CDN.

Il mio consiglio è di aspettare un po’; stiamo passando a Ember CLI, che sostituirà l’intera pipeline.

2 Mi Piace

@sam Quando passeremo a Ember CLI? Abbiamo lo stesso problema con l’installazione in sottocartella e Fastly CDN

Questa transizione è in corso: puoi già sviluppare in locale utilizzando Ember CLI, mancano ancora pochi mesi.

2 Mi Piace

Credo che questo dovrebbe essere risolto con ember-cli? @david sapresti dirmi se è così?

2 Mi Piace

Sì, ho apportato diversi miglioramenti alle source map nelle ultime settimane, che si applicano sia alle pipeline ‘legacy’ che a quelle Ember-CLI.

Ho appena controllato uno dei nostri siti ospitati in una sottocartella su tests-passed e le cose sembrano funzionare molto meglio :ok_hand:

5 Mi Piace

Questo argomento è stato chiuso automaticamente dopo 12 giorni. Non sono più consentite nuove risposte.