Merge pull request #1 from joeyparis/webpacker-manifest-asset-path

🐛 Fix bug that prevents loading assets with `webpacker-dev-server`
joeyparis · Nov 6, 2017 · 987c5c1 · 987c5c1
2 parents b3fac2c + 649b1b6
commit 987c5c1
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/lib/react/server_rendering/webpacker_manifest_container.rb b/lib/react/server_rendering/webpacker_manifest_container.rb
@@ -42,6 +42,8 @@ def find_asset(logical_path)
           asset_path = Webpacker.manifest.lookup(logical_path).to_s
           if Webpacker.dev_server.running?
             ds = Webpacker.dev_server
+            # Remove the protocol and host from the asset path. Sometimes webpacker includes this, sometimes it does not
+            asset_path.slice!("#{ds.protocol}://#{ds.host_with_port}")
             dev_server_asset = open("#{ds.protocol}://#{ds.host_with_port}#{asset_path}").read
             dev_server_asset.sub!(CLIENT_REQUIRE, '//\0')
             dev_server_asset