quilt_rails 3.4.0 → 3.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aa35ecd25ebbe9bd095f8e161debda741a45237533aef5d36751f9a1858dd5c7
-  data.tar.gz: 8de82016b228fe4f9ef5f21bd09380637db99d5b160345ce3910e7ca7fdf1e11
+  metadata.gz: aa72dc117e88bc8726ec6dcc658d86fe6882b62d91a964bac6267b351de1c595
+  data.tar.gz: d11b303d107674073e8a0bc597b59861ef627251c64554c515ef978f497cbe5e
 SHA512:
-  metadata.gz: 4e205902095208ed7d79c0e963389220b1c1d1050e46201935428b6949ca6f37f600e412b9cf1eeadf9435e7fd8fade00191edc922cbcb64819bb79acdc9af2f
-  data.tar.gz: f22a95a82bd538ff24ebaf4e15a1ada939b77e1be3e258c1f5a2ead5f342116517006934d625507f6770da5bce75e52675af5e48cea4c19ee948032c21a7342d
+  metadata.gz: b38651e9772a2a6882a9ab3064897e3f9c78ad75a05f9f2f6cedcb77602c59ba12581187c800694758dc841bdc87fde6a261d3c8516478639e58b9fe60f8263d
+  data.tar.gz: ef8e6e54e66d7e9132d738fb51efc5ccc06c4d7e243201ddd177fda4992e6260d1bb9b5790a7eeaa785c8f31f2e3f39f2cdec5119508ceae8fabd2e188a3cb85

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`.
 
@@ -171,7 +171,7 @@ In some cases you may want to send basic data from Rails to your React server. Q
 
 **Note:** The data passed should be data that is unlikely or will never change over the course of the session before they render any React components.
 
-**Note:** Please note the minimal dependencies require to use this feature is listed in [CHANGELOG](./CHANGELOG.md#[1.12.0]-2020-05-07)
+**Note:** Please note the minimal dependencies require to use this feature is listed in [CHANGELOG](./CHANGELOG.md#1.12.0-2020-05-07)
 
 ```ruby
 class ReactController < ApplicationController
@@ -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,7 +11,9 @@ 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
+      self.allow_integration_test = false
     end
 
     def mount?
@@ -19,11 +21,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

@@ -7,7 +7,9 @@ module Quilt
     include ReverseProxy::Controller
 
     def render_react(headers: {}, data: {})
-      raise DoNotIntegrationTestError if Rails.env.test?
+      if Rails.env.test? && !Quilt.configuration.allow_integration_test
+        raise DoNotIntegrationTestError
+      end
 
       # Allow concurrent loading to prevent this thread from blocking class
       # loading in controllers called by the Node server.
@@ -20,7 +22,8 @@ module Quilt
 
     def call_proxy(headers, data)
       if defined? ShopifySecurityBase
-        ShopifySecurityBase::HTTPHostRestriction.whitelist([Quilt.configuration.react_server_host]) do
+        allowlist = ShopifySecurityBase::HTTPHostRestriction.respond_to?(:allowlist) ? :allowlist : :whitelist
+        ShopifySecurityBase::HTTPHostRestriction.send(allowlist, [Quilt.configuration.react_server_host]) do
           proxy(headers, data)
         end
       else
@@ -30,10 +33,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 +45,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
@@ -53,8 +56,8 @@ module Quilt
     class ReactServerNoResponseError < StandardError
       def initialize(url)
         super(<<~MSG.squish)
-          Errno::ECONNREFUSED: Waiting for React server to boot up.
-          If this error persists verify that @shopify/react-server is configured on #{url}"
+          Connection refused while waiting for React server to boot up.
+          If this error persists, verify that @shopify/react-server is configured on #{url}.
         MSG
       end
     end
@@ -62,8 +65,10 @@ module Quilt
     class DoNotIntegrationTestError < StandardError
       def initialize
         super(<<~MSG.squish)
-          Do not try to use Rails integration tests on your quilt_rails app.
-          Instead use Jest and @shopify/react-testing to test your React application directly."
+          Please don't test React views with Ruby.
+          Jest and @shopify/react-testing should be used to test React components.
+
+          If you meant to query your Rails application, please check your integration test for errors.
         MSG
       end
     end

data/lib/quilt_rails/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Quilt
-  VERSION = "3.4.0"
+  VERSION = "3.5.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.5.1
 platform: ruby
 authors:
 - Mathew Allen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-08-26 00:00:00.000000000 Z
+date: 2021-08-30 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:
@@ -144,7 +157,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.2.20
 signing_key: 
 specification_version: 4
 summary: A turn-key solution for integrating server-rendered React into your Rails

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>