fix(#2236): Stripping the internals of Grape::Endpoint when NoMethodError is raised

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).
+* [#2368](https://github.com/ruby-grape/grape/pull/2368): Stripping the internals of `Grape::Endpoint` when `NoMethodError` is raised - [@jcagarcia](https://github.com/jcagarcia).
 * Your contribution here.
 
 ### 1.8.0 (2023/08/30)

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

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]