Schlechter Source-Map-Pfad bei Installation im Unterordner

Beitragen Fehler
1

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 „Gefällt mir“
2

@sam is the expert on source maps

3

@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 „Gefällt mir“
4

@sam Haben wir Lösungen für dieses Problem mit einer falschen Source Map? Ich habe das gleiche Problem bei der Installation in einem Unterordner und mit Fastly CDN.

5

Meine Empfehlung ist, einfach noch etwas zu warten. Wir wechseln zu Ember CLI, was diese gesamte Pipeline ersetzen wird.

2 „Gefällt mir“
6

@sam Wann wechseln wir zu Ember CLI? Wir haben das gleiche Problem mit der Installation in Unterordnern und dem Fastly CDN.

7

Dieser Schritt ist in Arbeit, du kannst bereits lokal mit Ember CLI entwickeln, in einigen Monaten ist es soweit.

2 „Gefällt mir“
9

Ich glaube, das sollte mit ember-cli sortiert werden? @david, wissen Sie, ob das der Fall ist?

2 „Gefällt mir“
10

Ja, ich habe in den letzten Wochen eine Reihe von Verbesserungen an Source Maps vorgenommen, die sowohl für ‘Legacy’- als auch für Ember-CLI-Pipelines gelten.

Ich habe gerade eine unserer gehosteten Unterordner-Websites auf tests-passed überprüft, und die Dinge scheinen viel besser zu funktionieren :ok_hand:

5 „Gefällt mir“
11

Dieses Thema wurde nach 12 Tagen automatisch geschlossen. Neue Antworten sind nicht mehr möglich.