From 9aea1c71d6527e0d253c0ac2690ff9fbf75a21ce Mon Sep 17 00:00:00 2001
From: Juan Carlos Garcia <juancarlos.garcia@flywire.com>
Date: Thu, 9 Nov 2023 23:30:27 +0100
Subject: [PATCH] fix(#2236): Stripping the internals of `Grape::Endpoint` when
 `NoMethodError` is raised

---
 CHANGELOG.md                |  1 +
 lib/grape/endpoint.rb       |  4 ++++
 spec/grape/endpoint_spec.rb | 11 +++++++++++
 3 files changed, 16 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index d17ce560c2..5425351a13 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -12,6 +12,7 @@
 
 * [#2364](https://github.com/ruby-grape/grape/pull/2364): Add missing requires - [@ericproulx](https://github.com/ericproulx).
 * [#2366](https://github.com/ruby-grape/grape/pull/2366): Default quality to 1.0 in the `Accept` header when omitted - [@hiddewie](https://github.com/hiddewie).
+* [#](): Stripping the internals of `Grape::Endpoint` when `NoMethodError` is raised - [@jcagarcia](https://github.com/jcagarcia).
 * Your contribution here.
 
 ### 1.8.0 (2023/08/30)
diff --git a/lib/grape/endpoint.rb b/lib/grape/endpoint.rb
index 663a3e8f1d..a25aa298ff 100644
--- a/lib/grape/endpoint.rb
+++ b/lib/grape/endpoint.rb
@@ -403,5 +403,9 @@ def options?
       options[:options_route_enabled] &&
         env[Grape::Http::Headers::REQUEST_METHOD] == Grape::Http::Headers::OPTIONS
     end
+
+    def method_missing(name, *args)
+      raise NoMethodError.new("undefined method `#{name}` for #{self.class}")
+    end
   end
 end
diff --git a/spec/grape/endpoint_spec.rb b/spec/grape/endpoint_spec.rb
index 8cdcc1cffe..e3de82a7a0 100644
--- a/spec/grape/endpoint_spec.rb
+++ b/spec/grape/endpoint_spec.rb
@@ -692,6 +692,17 @@ def app
     end
   end
 
+  describe '#method_missing' do
+    it 'raises NoMethodError but stripping the internals of the Grape::Endpoint class' do
+      subject.get('/hey') do
+        undefined_helper
+      end
+      expect {
+        get '/hey'
+      }.to raise_error(NoMethodError, /^undefined method `undefined_helper` for #<Class:0x[0-9a-fA-F]+>$/)
+    end
+  end
+
   it 'does not persist params between calls' do
     subject.post('/new') do
       params[:text]