quilt_rails 3.4.0 → 3.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aa35ecd25ebbe9bd095f8e161debda741a45237533aef5d36751f9a1858dd5c7
-  data.tar.gz: 8de82016b228fe4f9ef5f21bd09380637db99d5b160345ce3910e7ca7fdf1e11
+  metadata.gz: 0a6933fd12f6ae25d25f4afab855ba022287ac3c2e353d3b1d1aa2764adb6e40
+  data.tar.gz: 88e908022a12e1ed55bcd8859bc7be736e9d844832ed7df30f81caf3f822c875
 SHA512:
-  metadata.gz: 4e205902095208ed7d79c0e963389220b1c1d1050e46201935428b6949ca6f37f600e412b9cf1eeadf9435e7fd8fade00191edc922cbcb64819bb79acdc9af2f
-  data.tar.gz: f22a95a82bd538ff24ebaf4e15a1ada939b77e1be3e258c1f5a2ead5f342116517006934d625507f6770da5bce75e52675af5e48cea4c19ee948032c21a7342d
+  metadata.gz: b2e02a1233697cb16ef35ec291de7736b3e0721d516bcd566c309c88704e996fc5ce40c3ccb72abc54c6dbf1b2fbe384c41bf333a760e1e889b7ee280331f5ba
+  data.tar.gz: 506bfd185f702ad39e7f6d21713524b002bc90e45e0e1aaed0832d6aee2f5d0d998a104c46b9272ab001040fd1ed49be41a788107cc9c0fea9145697dffb10ff

data/README.md

@@ -1,6 +1,6 @@
 # quilt_rails
 
-A turn-key solution for integrating Quilt client-side libraries into your Rails app, with support for server-side-rendering using [`@shopify/react-server`](https://www.npmjs.com/package/@shopify/react-server), integration with [`@shopify/sewing-kit`](https://github.com/Shopify/sewing-kit) for building, testing and linting, and front-end performance tracking through [`@shopify/performance`](https://www.npmjs.com/package/@shopify/performance).
+A turn-key solution for integrating [Quilt](https://github.com/Shopify/quilt) client-side libraries into your Rails app, with support for server-side-rendering using [`@shopify/react-server`](https://www.npmjs.com/package/@shopify/react-server), integration with [`@shopify/sewing-kit`](https://github.com/Shopify/sewing-kit) for building, testing and linting, and front-end performance tracking through [`@shopify/performance`](https://www.npmjs.com/package/@shopify/performance).
 
 ## Table of Contents
 
@@ -67,9 +67,9 @@ Follow [this guide](./docs/manual-installation.md) on how to do manual setup wit
 
 For fast tests with consistent results, test front-end components using the tools provided by sewing-kit instead of Rails integration tests.
 
-Use [`sewing-kit test`](https://github.com/Shopify/sewing-kit/blob/master/docs/commands/test.md#L3) to run all `.test.{js|ts}x` files in the `app/ui` directory. [Jest](https://jestjs.io/) is used as a test runner, with customization available via [its sewing-kit plugin](https://github.com/Shopify/sewing-kit/blob/master/docs/plugins/jest.md).
+Use [`sewing-kit test`](https://github.com/Shopify/sewing-kit/blob/main/docs/commands/test.md#L3) to run all `.test.{js|ts}x` files in the `app/ui` directory. [Jest](https://jestjs.io/) is used as a test runner, with customization available via [its sewing-kit plugin](https://github.com/Shopify/sewing-kit/blob/main/docs/plugins/jest.md).
 
-For testing React applications we provide and support [`@shopify/react-testing`](https://github.com/Shopify/quilt/tree/master/packages/react-testing).
+For testing React applications we provide and support [`@shopify/react-testing`](https://github.com/Shopify/quilt/tree/main/packages/react-testing).
 
 ##### Example
 
@@ -102,11 +102,11 @@ describe('MyComponent', () => {
 
 Often you will want to hook up custom polyfills, global mocks, or other logic that needs to run either before the initialization of the test environment, or once for each test suite.
 
-By default, sewing-kit will look for such test setup files under `/app/ui/tests`. Check out the [documentation](https://github.com/Shopify/sewing-kit/blob/master/docs/plugins/jest.md#smart-defaults) for more details.
+By default, sewing-kit will look for such test setup files under `/app/ui/tests`. Check out the [documentation](https://github.com/Shopify/sewing-kit/blob/main/docs/plugins/jest.md#smart-defaults) for more details.
 
 ##### Interacting with the request and response in React code
 
-React-server sets up [@shopify/react-network](https://github.com/Shopify/quilt/blob/master/packages/react-network) automatically, so most interactions with the request or response can be done from inside the React app.
+React-server sets up [@shopify/react-network](https://github.com/Shopify/quilt/blob/main/packages/react-network) automatically, so most interactions with the request or response can be done from inside the React app.
 
 ##### Example: getting headers
 
@@ -131,7 +131,7 @@ function App() {
 export default App;
 ```
 
-**Note:** This solution works out of the box for initial server-side renders. If you wish to have consistent access to request headers on subsequent client-side renders, take a look at [`NetworkUniversalProvider`](https://github.com/Shopify/quilt/tree/master/packages/react-network#networkuniversalprovider).
+**Note:** This solution works out of the box for initial server-side renders. If you wish to have consistent access to request headers on subsequent client-side renders, take a look at [`NetworkUniversalProvider`](https://github.com/Shopify/quilt/tree/main/packages/react-network#networkuniversalprovider).
 
 ##### Example: sending custom headers from Rails controller
 
@@ -147,7 +147,7 @@ class ReactController < ApplicationController
 end
 ```
 
-🗒️ if you don't have a controller. Follow the [instruction](./docs/manual-installation#add-a-react-controller-and-routes) to setup `quilt_rails` in a controller instead of using the engine.
+🗒️ if you don't have a controller. Follow the [instruction](./docs/manual-installation.md#option-2-add-a-react-controller-and-routes) to setup `quilt_rails` in a controller instead of using the engine.
 
 Headers can be accessed during server-side-rendering with the `useRequestHeader` hook from `@shopify/react-network`.
 
@@ -183,7 +183,7 @@ class ReactController < ApplicationController
 end
 ```
 
-🗒️ if you don't have a controller. Follow the [instruction](./docs/manual-installation#add-a-react-controller-and-routes) to setup `quilt_rails` in a controller instead of using the engine.
+🗒️ if you don't have a controller. Follow the [instruction](./docs/manual-installation.md#option-2-add-a-react-controller-and-routes) to setup `quilt_rails` in a controller instead of using the engine.
 
 If using `react-server` without a customized server & client file, this will be automatically passed into your application as the `data` prop. If `react-server` is not being used or a customized server / client file was provided, check out [`react-server/webpack-plugin`](../../packages/react-server/src/webpack-plugin/webpack-plugin.ts) on how to pass the data to React.
 
@@ -222,9 +222,9 @@ export default App;
 
 #### Isomorphic state
 
-With SSR enabled React apps, state must be serialized on the server and deserialized on the client to keep it consistent. When using `@shopify/react-server`, the best tool for this job is [`@shopify/react-html`](https://github.com/Shopify/quilt/tree/master/packages/react-html)'s [`useSerialized`](https://github.com/Shopify/quilt/tree/master/packages/react-html#in-your-application-code) hook.
+With SSR enabled React apps, state must be serialized on the server and deserialized on the client to keep it consistent. When using `@shopify/react-server`, the best tool for this job is [`@shopify/react-html`](https://github.com/Shopify/quilt/tree/main/packages/react-html)'s [`useSerialized`](https://github.com/Shopify/quilt/tree/main/packages/react-html#in-your-application-code) hook.
 
-`useSerialized` can be used to implement [universal-providers](https://github.com/Shopify/quilt/tree/master/packages/react-universal-provider#what-is-a-universal-provider-), allowing application code to manage what is persisted between the server and client without adding any custom code to client or server entrypoints. We offer some for common use cases such as [GraphQL](https://github.com/Shopify/quilt/tree/master/packages/react-graphql-universal-provider), and [I18n](https://github.com/Shopify/quilt/tree/master/packages/react-i18n-universal-provider).
+`useSerialized` can be used to implement [universal-providers](https://github.com/Shopify/quilt/tree/main/packages/react-universal-provider#what-is-a-universal-provider-), allowing application code to manage what is persisted between the server and client without adding any custom code to client or server entrypoints. We offer some for common use cases such as [GraphQL](https://github.com/Shopify/quilt/tree/main/packages/react-graphql-universal-provider), and [I18n](https://github.com/Shopify/quilt/tree/main/packages/react-i18n-universal-provider).
 
 #### Customizing the Node server
 
@@ -289,7 +289,7 @@ end
 
 #### Exception monitoring with Bugsnag
 
-For an opinionated universal Bugsnag+React setup we provide and support [`@shopify/react-bugsnag`](https://github.com/Shopify/quilt/tree/master/packages/react-bugsnag).
+For an opinionated universal Bugsnag+React setup we provide and support [`@shopify/react-bugsnag`](https://github.com/Shopify/quilt/tree/main/packages/react-bugsnag).
 
 ##### Example
 

data/app/controllers/quilt/ui_controller.rb

@@ -5,12 +5,17 @@ module Quilt
     include Quilt::ReactRenderable
     layout(false)
 
-    rescue_from(Quilt::ReactRenderable::ReactServerNoResponseError) do
-      render(:react_render_error, status: :internal_server_error)
-    end
-
     def index
       render_react
+    rescue Quilt::ReactRenderable::ReactServerNoResponseError
+      sleep(1)
+      retry if execution_count < 10
+      raise
+    end
+
+    def execution_count
+      @times ||= 0
+      @times = @times.next
     end
   end
 end

data/lib/quilt_rails/configuration.rb

@@ -11,6 +11,7 @@ module Quilt
 
       self.react_server_host     = "#{react_server_ip}:#{react_server_port}"
       self.react_server_protocol = ENV['REACT_SERVER_PROTOCOL'] || "http"
+      self.logger                = ::Logger.new($stdout)
       self.mount                 = true
     end
 
@@ -19,11 +20,13 @@ module Quilt
     end
   end
 
-  def self.configuration
-    @configuration ||= Configuration.new
-  end
+  class << self
+    def configuration
+      @configuration ||= Configuration.new
+    end
 
-  def self.configure
-    yield(configuration)
+    def configure
+      yield(configuration)
+    end
   end
 end

data/lib/quilt_rails/engine.rb

@@ -6,6 +6,10 @@ module Quilt
 
     config.quilt = Quilt.configuration
 
+    initializer(:initialize_quilt_logger, after: :initialize_logger) do
+      config.quilt.logger = ::Rails.logger
+    end
+
     initializer(:mount_quilt, before: :add_builtin_route) do |app|
       if config.quilt.mount?
         app.routes.append do

data/lib/quilt_rails/logger.rb

@@ -3,11 +3,14 @@
 module Quilt
   module Logger
     def self.log(string)
-      if Rails.logger.nil?
-        puts string
-      else
-        Rails.logger.info(string)
-      end
+      ActiveSupport::Deprecation.warn(<<~MSG.squish)
+        Quilt::Logger.log is deprecated. Please use Quilt.logger instead.
+      MSG
+      Quilt.configuration.logger.info(string)
     end
   end
+
+  class << self
+    delegate(:logger, to: :configuration)
+  end
 end

data/lib/quilt_rails/react_renderable.rb

@@ -30,10 +30,10 @@ module Quilt
 
     def proxy(headers, data)
       url = "#{Quilt.configuration.react_server_protocol}://#{Quilt.configuration.react_server_host}"
-      Quilt::Logger.log("[ReactRenderable] proxying to React server at #{url}")
+      Quilt.logger.info("[ReactRenderable] proxying to React server at #{url}")
 
       unless headers.blank?
-        Quilt::Logger.log("[ReactRenderable] applying custom headers #{headers.inspect}")
+        Quilt.logger.info("[ReactRenderable] applying custom headers #{headers.inspect}")
       end
 
       begin
@@ -42,7 +42,7 @@ module Quilt
           headers: headers.merge('X-Request-ID': request.request_id, 'X-Quilt-Data': data.to_json)
         ) do |callbacks|
           callbacks.on_response do |status_code, _response|
-            Quilt::Logger.log("[ReactRenderable] #{url} returned #{status_code}")
+            Quilt.logger.info("[ReactRenderable] #{url} returned #{status_code}")
           end
         end
       rescue Errno::ECONNREFUSED

data/lib/quilt_rails/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Quilt
-  VERSION = "3.4.0"
+  VERSION = "3.4.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: quilt_rails
 version: !ruby/object:Gem::Version
-  version: 3.4.0
+  version: 3.4.1
 platform: ruby
 authors:
 - Mathew Allen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-08-26 00:00:00.000000000 Z
+date: 2021-02-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 3.2.0
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.2.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.2.0
 - !ruby/object:Gem::Dependency
   name: rails-reverse-proxy
   requirement: !ruby/object:Gem::Requirement
@@ -92,7 +106,6 @@ files:
 - Rakefile
 - app/controllers/quilt/performance_report_controller.rb
 - app/controllers/quilt/ui_controller.rb
-- app/views/quilt/ui/react_render_error.html
 - config/routes.rb
 - lib/generators/quilt/USAGE
 - lib/generators/quilt/install_generator.rb
@@ -124,7 +137,7 @@ files:
 - lib/quilt_rails/react_renderable.rb
 - lib/quilt_rails/trusted_ui_server_csrf_strategy.rb
 - lib/quilt_rails/version.rb
-homepage: https://github.com/Shopify/quilt/tree/master/gems/quilt_rails
+homepage: https://github.com/Shopify/quilt/tree/main/gems/quilt_rails
 licenses:
 - MIT
 metadata:

data/app/views/quilt/ui/react_render_error.html

@@ -1,21 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-  <title>React Render Error</title>
-  <meta name="viewport" content="width=device-width,initial-scale=1">
-  <meta http-equiv="refresh" content="3;URL='/'" />
-  <link rel="stylesheet" href="https://unpkg.com/@shopify/polaris@5.1.0/dist/styles.css" />
-</head>
-<body>
-  <div>
-    <div>
-      <h1>Waiting for React Sever to boot up.</h1>
-      <p>This page will refresh automatically.</p>
-    </div>
-    <p>
-      If this error persists, ensure <code>@shopify/react-server</code>
-      is running on <code>http://localhost:8081</code>.
-    </p>
-  </div>
-</body>
-</html>