Fix ruby-grape#249: leaking of options and internals in default seria…

…lization.

CHANGELOG.md

@@ -7,6 +7,7 @@
 
 #### Fixes
 
+* [#249](https://github.com/ruby-grape/grape-entity/issues/249): Fix leaking of options and internals in default serialization - [@dblock](https://github.com/dblock), [@KingsleyKelly](https://github.com/KingsleyKelly).
 * [#248](https://github.com/ruby-grape/grape-entity/pull/248): Fix `nil` values causing errors when `merge` option passed - [@arempe93](https://github.com/arempe93).
 * Your contribution here.
 

lib/grape_entity/entity.rb

@@ -424,6 +424,11 @@ def presented
       end
     end
 
+    # Prevent default serialization of :options or :delegator.
+    def inspect
+      "#<#{self.class.name}:#{self.object_id} @object=#{@object}>"
+    end
+
     def initialize(object, options = {})
       @object = object
       @delegator = Delegator.new object

spec/grape_entity/entity_spec.rb

@@ -1347,6 +1347,13 @@ class NoPathCharacterEntity < Grape::Entity
       end
     end
 
+    describe "#inspect" do
+      it 'does not serialize delegator or options' do
+        expect(subject.inspect).to_not include "@options"
+        expect(subject.inspect).to_not include "@delegator"
+      end
+    end
+
     describe '#value_for' do
       before do
         fresh_class.class_eval do