forest_admin_rails 1.12.0 → 1.12.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 12744bc9381af2c455a8ab1dd56e68443bc813d6f5df92f2b0f56a6fce4d3fa8
-  data.tar.gz: d080d3ccbb57fecc113f6c0b41d871f115fc80e06dcb87a4fc6e0ee99fa1afc3
+  metadata.gz: '099b5f8e72f047e94d152def7e081f05321c7bf0b1aaa75f0484c917acd7fc61'
+  data.tar.gz: 0af28ac77c06cbed7cb5212c244232a1f3a3e42aa64a49f02c34281f821d61a9
 SHA512:
-  metadata.gz: 903af7b0abb45535e2e5e56bf358f8d3103f4518f48a0ff79b1bad9157bcd3ae19f5f323ff8a0530905deab1864f52034e2fc23403236fc612d4915e60328b13
-  data.tar.gz: 34302839a5df138d04551410c2031764f1fee00be5fda460ae2a3d3cee9c089da6303e6cd5d220665175b405a17f5c3404d5317499607cbc9b0e94830007b94a
+  metadata.gz: c1c0422ec8739e19267569261e0afe16e1cca5f1673cbf65ceab6142000c9df32b96235745fad440b16e67d4f9751a1b1785ed6b15fde86f404f4e62dd09c973
+  data.tar.gz: ab902ad4d0e18a6f5dd332f9e358f7d018208f7c931984f86b7ebeafe0b3717c23058f2a723adfbe32ccfc599df483d3bb67ab54535e17b1d0720489f9eed043

data/app/controllers/forest_admin_rails/forest_controller.rb

@@ -7,8 +7,8 @@ module ForestAdminRails
     skip_forgery_protection
 
     def index
-      if ForestAdminAgent::Http::Router.routes.key? params['route_alias']
-        route = ForestAdminAgent::Http::Router.routes[params['route_alias']]
+      if ForestAdminAgent::Http::Router.cached_routes.key? params['route_alias']
+        route = ForestAdminAgent::Http::Router.cached_routes[params['route_alias']]
 
         begin
           forest_response route[:closure].call({ params: params.to_unsafe_h, headers: request.headers.to_h })

data/config/initializers/forest_admin_route_cache.rb

@@ -0,0 +1,29 @@
+if defined?(ForestAdminAgent::Http::Router)
+  cache_disabled = ForestAdminAgent::Http::Router.cache_disabled?
+
+  if cache_disabled
+    Rails.logger.warn('[ForestAdmin] Route caching is DISABLED - this will impact performance')
+    Rails.logger.warn('[ForestAdmin] To enable caching, remove disable_route_cache: true from configuration')
+  else
+    Rails.application.config.after_initialize do
+      start_time = Time.now
+      routes = ForestAdminAgent::Http::Router.cached_routes
+      elapsed = ((Time.now - start_time) * 1000).round(2)
+
+      Rails.logger.info(
+        "[ForestAdmin] Successfully pre-cached #{routes.size} routes in #{elapsed}ms"
+      )
+    rescue StandardError => e
+      Rails.logger.error(
+        "[ForestAdmin] CRITICAL: Failed to pre-cache routes: #{e.class} - #{e.message}"
+      )
+      Rails.logger.error(e.backtrace.first(10).join("\n"))
+
+      raise e if Rails.env.production?
+
+      Rails.logger.warn(
+        '[ForestAdmin] Routes will be computed on first request (performance degradation)'
+      )
+    end
+  end
+end

data/config/routes.rb

@@ -1,15 +1,20 @@
 ForestAdminRails::Engine.routes.draw do
-  Rails.error.handle(ForestAdminDatasourceToolkit::Exceptions::ForestException) do
-    scope '/forest' do
-      ForestAdminAgent::Http::Router.routes.each do |name, agent_route|
-        match agent_route[:uri],
-              defaults: { format: agent_route[:format] },
-              to: 'forest#index',
-              via: agent_route[:method],
-              as: name,
-              route_alias: name,
-              constraints: { id: /[a-zA-Z0-9\.]*+/ }
-      end
+  scope '/forest' do
+    # Use cached_routes to avoid recomputing routes during Rails route initialization
+    ForestAdminAgent::Http::Router.cached_routes.each do |name, agent_route|
+      match agent_route[:uri],
+            defaults: { format: agent_route[:format] },
+            to: 'forest#index',
+            via: agent_route[:method],
+            as: name,
+            route_alias: name,
+            constraints: { id: /[a-zA-Z0-9\.]*+/ }
     end
+  rescue StandardError => e
+    if defined?(Rails) && Rails.logger
+      Rails.logger.error("[ForestAdmin] CRITICAL: Failed to initialize routes: #{e.class} - #{e.message}")
+      Rails.logger.error(e.backtrace.join("\n"))
+    end
+    raise e
   end
 end

data/lib/forest_admin_rails/version.rb

@@ -1,3 +1,3 @@
 module ForestAdminRails
-  VERSION = "1.12.0"
+  VERSION = "1.12.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: forest_admin_rails
 version: !ruby/object:Gem::Version
-  version: 1.12.0
+  version: 1.12.1
 platform: ruby
 authors:
 - Matthieu
@@ -68,6 +68,7 @@ files:
 - app/controllers/forest_admin_rails/forest_controller.rb
 - app/helpers/forest_admin_rails/application_helper.rb
 - config/initializers/forest_admin_error_subscriber.rb
+- config/initializers/forest_admin_route_cache.rb
 - config/routes.rb
 - lib/forest_admin_rails.rb
 - lib/forest_admin_rails/engine.rb