サブフォルダへのインストールでソースマップのパスが不適切

貢献 バグ
1

サブフォルダへのインストールでは、Firefox コンソールでソースマップが見つからないというエラーが繰り返し発生していました。

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

ご覧の通り、ソースマップの URL から /community というパス接頭辞が欠落しています。

そこで少し調査してみました。

lib/tasks/assets.rake は以下のように記述されています：

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

一方、lib/global_path.rb 内の cdn_relative_path は次のようになっています。

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

ご覧の通り、cdn_path と upload_cdn_path はどちらも最終的に path メソッドを呼び出し、サブフォルダの接頭辞が正しく付加されるようにしています。しかし、cdn_relative_path は path メソッドを呼び出さず、単に引数（path という名前のパラメータ）を返すだけです。

余談ですが、cdn_path の三項演算子の中間部分でも path(p) を返す必要があるのではないかと考えています。

「いいね！」 1
2

@sam はソースマップの専門家です

3

@david は過去にこれいじくったことがありますが、CDN とサブフォルダのパスを正しく設定するのは簡単ではありません。現在動作している箇所に後退をもたらさず、あなたの特別な組み合わせに特化したものであれば、PR を歓迎します。

「いいね！」 3
4

@sam このソースマップが正しくないという問題に対する解決策はありますか？私はサブフォルダインストールとFastly CDNでも同じ問題に直面しています。

5

私の推奨は、しばらく待ってから行うことです。私たちは Ember CLI へ移行しており、これがこのパイプライン全体を置き換えることになります。

「いいね！」 2
6

@sam エMBER CLIへの移行はいつ予定されていますか？サブフォルダへのインストールとFastly CDNにおいて、同じ問題が発生しています。

7

この移行は進行中です。Ember CLI を使ってローカル環境で開発することはすでに可能ですが、本番環境への移行にはあと数ヶ月かかります。

「いいね！」 2
9

これは ember-cli で解決されるべきだと思いますか？ @david ご存知でしたら教えていただけますか？

「いいね！」 2
10

ここ数週間、ソースマップの改善をいくつか行いました。これは「レガシー」と Ember-CLI の両方のパイプラインに適用されます。

テストに合格したホストされているサブフォルダーサイトの 1 つを確認したところ、物事ははるかにうまく機能しているようです :ok_hand:

「いいね！」 5
11

このトピックは12日後に自動的に閉じられました。返信はもう許可されていません。