twirp-on-rails 1.2.0 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 308f1b4a611dd3496ffa026dae81410bffedffb0a2330ef009c4090b2ce71c24
-  data.tar.gz: dcfba3e14253882f6b7ba0943a89876789fb0d8e68015e397c6143e5417f7d35
+  metadata.gz: 4525c9180d9b5ee9316453fefadc8982793f5aee3ebe55db5136fb2970e3b764
+  data.tar.gz: 68cb9924021caeefa024616b0d68d28a4a20ec77bf663b658be973fbb9fea86b
 SHA512:
-  metadata.gz: 140d92258a215e8de86f641a39aeb7cf6e07034ee3c15e9ae187ed9a8f7d1d45d360a7240ecdc0599069dbe63558de7f2ba9ce15ecf7d7c538968bd96f784596
-  data.tar.gz: 3dcfdf03ddab84f16a8b8a052439b22e982456a7076c2ac54d2580244c16846b17cf70709a72bfa4a0be42743f139624bf338a97ff454d9fba83455be77eef90
+  metadata.gz: 545989b2edaa3b4e78291f6847042e0728053e677b2e144e5b0463bdc5e00acd86fd0ea95ff8de6ae3ff19a12ac99d3941e325a27188a2f2fb0e4b23e787d9f4
+  data.tar.gz: 9075fd125690021fa6c1f192f0474fcc53639677cfa4c08904a20e7d31c1fbd487773ac227d42200c3e7e894b4ba9605486b017400d64c6c12bdd8cc455e96fc

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 ## [Unreleased]
 
+## [1.4.0] - 2025-01-28
+
+- Fix middleware to not apply where it isn't expected. 
+
+## [1.3.0] - 2025-01-20
+
+- Add some logging.
 
 ## [1.2.0] - 2024-10-04
 

data/Gemfile

@@ -9,7 +9,11 @@ gem "rake"
 
 gem "debug"
 gem "rspec-rails"
-gem "sqlite3", "~> 1.4"
 gem "standard", ">= 1.35.1"
 gem "standard-performance"
 gem "standard-rails"
+
+# These standard library gems need to be here for with certain Rails 7/Ruby 3.4 combinations. Delete eventually.
+gem "mutex_m"
+gem "bigdecimal"
+gem "drb"

data/README.md

@@ -151,6 +151,22 @@ As an Engine, we avoid all the standard Rails middleware. That's nice for simpli
 Rails.application.config.twirp.middleware = [Rack::Deflater]
 ```
 
+### Logging
+
+Our built-in logging outputs the result of each request. 
+
+You could replace our logger if you want different output: 
+
+```ruby
+Rails.application.config.twirp.logger = Rack::CommonLogger
+```
+
+Additionally, you can log the full Twirp response object to help with debugging: 
+
+```ruby
+Rails.application.config.twirp.verbose_logging = true
+```
+
 ## Bonus Features
 
 Outside the [Twirp spec](https://twitchtv.github.io/twirp/docs/spec_v7.html), we have some (optional) extra magic. They might be useful to you, but you can easily ignore them too.
@@ -170,6 +186,8 @@ Rails.application.config.twirp.middleware = [
 ]
 ```
 
+Note: The Handler will still be run, but you won't need to send back the response. Make sure your RPC is idempotent! Future versions hope to make it easier to short-circuit expensive parts of the handler. 
+
 ## TODO
 
 * More docs!

data/lib/twirp/rails/configuration.rb

@@ -9,6 +9,15 @@ module Twirp
       # Where to mount twirp routes. Defaults to /twirp
       attr_accessor :endpoint
 
+      # Logger to use for Twirp requests. Defaults to Rails.logger
+      attr_accessor :logger
+
+      # Whether to log full Twirp responses. Can be useful for debugging, but can expose sensitive data.
+      # Defauts to false
+      # Example:
+      #   Twirp Response: <Twirp::Example::Haberdasher::Hat: inches: 24, color: "Tan", name: "Pork Pie">
+      attr_accessor :verbose_logging
+
       # An array of directories to search for *_twirp.rb files
       # Defaults to ["lib"]
       attr_accessor :load_paths
@@ -25,6 +34,8 @@ module Twirp
         @auto_mount = false
         @endpoint = "/twirp"
         @load_paths = ["lib"]
+        @logger = Logger
+        @verbose_logging = false
         @middleware = []
         @service_hooks = {}
       end

data/lib/twirp/rails/engine.rb

@@ -26,8 +26,15 @@ module Twirp
           app.config.twirp.send(key)
         end
 
-        app.config.twirp.middleware.each do |middleware|
-          app.config.middleware.use middleware
+        # Set up logging
+        middleware.use app.config.twirp.logger, ::Rails.logger
+        app.config.twirp.middleware.each do |user_middleware|
+          middleware.use user_middleware
+        end
+
+        # Load all Twirp files
+        app.config.twirp.load_paths.each do |directory|
+          ::Rails.root.glob("#{directory}/**/*_twirp.rb").sort.each { |file| require file }
         end
       end
     end
@@ -35,17 +42,29 @@ module Twirp
     class << self
       def services
         if @services.nil?
-          ::Rails.application.config.twirp.load_paths.each do |directory|
-            ::Rails.root.glob("#{directory}/**/*_twirp.rb").sort.each { |file| require file }
-          end
-
           @services = Twirp::Service.subclasses.map(&:new)
 
           # Install hooks that may be defined in the config
           @services.each do |service|
+            # Add user-defined hooks
             ::Rails.application.config.twirp.service_hooks.each do |hook_name, hook|
               service.send(hook_name, &hook)
             end
+
+            # Add our own logging hooks
+            service.on_success do |env|
+              if ::Rails.application.config.twirp.verbose_logging
+                ::Rails.logger.debug { "Twirp Response: #{env[:output].inspect}" }
+              end
+            end
+
+            service.on_error do |error, _env|
+              ::Rails.logger.debug { "Twirp Response: #{error.inspect}" }
+            end
+
+            service.exception_raised do |exception, _env|
+              ::Rails.logger.error { "Twirp Exception (#{exception.class}: #{exception.message})\n#{exception.backtrace.join("\n")}" }
+            end
           end
         end
 
@@ -54,11 +73,3 @@ module Twirp
     end
   end
 end
-
-class Twirp::Service
-  # Override inspect to show all available RPCs
-  # This is used when displaying routes.
-  def inspect
-    self.class.rpcs.map { |rpc| "#{self.class.name.demodulize.underscore}_handler##{rpc[1][:ruby_method]}" }.join("\n")
-  end
-end

data/lib/twirp/rails/error.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module Twirp
+  module Rails
+    class Error < StandardError
+    end
+  end
+end

data/lib/twirp/rails/handler.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative "rescuable" # Ruby 2.7 fails without this. Remove eventually.
+
 module Twirp
   module Rails
     class Handler

data/lib/twirp/rails/logger.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+# Rack::CommonLogger is nice but we can do better.
+# Rails doesn't use it, but we need to log Twirp requests.
+# Here's an example from Rack::CommonLogger:
+#   127.0.0.1 - - [12/Jan/2025:17:09:49 -0500] "POST /twirp/twirp.example.haberdasher.Haberdasher/MakeHat HTTP/1.0" 200 - 439.0060
+#
+# Rails gives us this:
+#   Started POST "/twirp/twirp.example.haberdasher.Haberdasher/MakeHat" for 127.0.0.1 at 2025-01-12 22:48:00 -0500
+# but we also want to know the result of the Twirp call.
+# Here's what this Logger adds:
+#   Twirp 200 in 2ms as application/protobuf
+#
+
+module Twirp
+  module Rails
+    class Logger < ::Rack::CommonLogger
+      private
+
+      def log(env, status, response_headers, began_at)
+        content_type = response_headers["content-type"].presence
+        content_encoding = response_headers["content-encoding"].presence
+        @logger.info { "Twirp #{status} in #{duration_in_ms(began_at)}ms#{" as #{content_type}" if content_type}#{" with content-encoding: #{content_encoding}" if content_encoding}" }
+      end
+
+      def duration_in_ms(time)
+        ((::Rack::Utils.clock_time - time) * 1000).to_i
+      end
+    end
+  end
+end

data/lib/twirp/rails/version.rb

@@ -2,6 +2,6 @@
 
 module Twirp
   module Rails
-    VERSION = "1.2.0"
+    VERSION = "1.4.0"
   end
 end

data/lib/twirp/rails.rb

@@ -1,31 +1,30 @@
 # frozen_string_literal: true
 
-require_relative "rails/version"
+require "twirp"
+require "active_support/notifications"
 
-module Twirp
-  module Rails
-    class Error < StandardError; end
-    # Your code goes here...
-  end
+require "zeitwerk"
+loader = Zeitwerk::Loader.for_gem_extension(Twirp)
+loader.ignore("#{__dir__}/on/rails.rb")
+loader.setup
+
+module Twirp::Rails
 end
 
-require "twirp"
-require "active_support/notifications"
-require_relative "rails/callbacks"
-require_relative "rails/configuration"
-require_relative "rails/dispatcher"
-require_relative "rails/engine"
-require_relative "rails/rescuable"
-require_relative "rails/handler"
+loader.eager_load
+
+Twirp::Service.class_eval do
+  # Override initialize to make handler argument optional.
+  # When left nil, we will use our dispatcher.
+  alias_method :original_initialize, :initialize
+  def initialize(handler = nil)
+    handler ||= Twirp::Rails::Dispatcher.new(self.class)
+    original_initialize(handler)
+  end
 
-module Twirp
-  class Service
-    # Override initialize to make handler argument optional.
-    # When left nil, we will use our dispatcher.
-    alias_method :original_initialize, :initialize
-    def initialize(handler = nil)
-      handler ||= Twirp::Rails::Dispatcher.new(self.class)
-      original_initialize(handler)
-    end
+  # Override inspect to show all available RPCs
+  # This is used when displaying routes.
+  def inspect
+    self.class.rpcs.map { |rpc| "#{self.class.name.demodulize.underscore}_handler##{rpc[1][:ruby_method]}" }.join("\n")
   end
 end

metadata

@@ -1,15 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: twirp-on-rails
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.4.0
 platform: ruby
 authors:
 - Daniel Morrison
 - Darron Schall
-autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-10-04 00:00:00.000000000 Z
+date: 2025-01-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -61,7 +60,9 @@ files:
 - lib/twirp/rails/configuration.rb
 - lib/twirp/rails/dispatcher.rb
 - lib/twirp/rails/engine.rb
+- lib/twirp/rails/error.rb
 - lib/twirp/rails/handler.rb
+- lib/twirp/rails/logger.rb
 - lib/twirp/rails/rack/conditional_post.rb
 - lib/twirp/rails/rescuable.rb
 - lib/twirp/rails/version.rb
@@ -74,7 +75,6 @@ metadata:
   homepage_uri: https://github.com/collectiveidea/twirp-rails
   source_code_uri: https://github.com/collectiveidea/twirp-rails
   changelog_uri: https://github.com/collectiveidea/twirp-rails/blob/main/CHANGELOG.md
-post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -89,8 +89,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.16
-signing_key: 
+rubygems_version: 3.6.2
 specification_version: 4
 summary: Use Twirp RPC with Rails
 test_files: []