Merge pull request #1691 from doorkeeper-gem/make-errors-backward-com…

…patible

Make new errors backward compatible

CHANGELOG.md

@@ -8,6 +8,7 @@ User-visible changes worth mentioning.
 ## main
 
 - [#PR ID] Add your changelog here.
+- [#1691] Make new Doorkeeper errors backward compatible with older extensions.
 
 ## 5.6.8
 

lib/doorkeeper/oauth/error_response.rb

@@ -10,14 +10,26 @@ class ErrorResponse < BaseResponse
       def self.from_request(request, attributes = {})
         new(
           attributes.merge(
-            name: request.error&.name_for_response,
-            exception_class: request.error,
+            name: error_name_for(request.error),
+            exception_class: exception_class_for(request.error),
             state: request.try(:state),
             redirect_uri: request.try(:redirect_uri),
           ),
         )
       end
 
+      def self.error_name_for(error)
+        error.respond_to?(:name_for_response) ? error.name_for_response : error
+      end
+
+      def self.exception_class_for(error)
+        return error if error.respond_to?(:name_for_response)
+
+        "Doorkeeper::Errors::#{error.to_s.classify}".safe_constantize
+      end
+
+      private_class_method :error_name_for, :exception_class_for
+
       delegate :name, :description, :state, to: :@error
 
       def initialize(attributes = {})

spec/lib/oauth/error_response_spec.rb

@@ -36,6 +36,13 @@
       error = described_class.from_request double(error: Doorkeeper::Errors::InvalidClient, state: :hello)
       expect(error.state).to eq(:hello)
     end
+
+    it "supports old extensions" do
+      error = described_class.from_request double(error: :invalid_client)
+      expect(error.name).to eq(:invalid_client)
+
+      expect { error.raise_exception! }.to raise_error(Doorkeeper::Errors::InvalidClient)
+    end
   end
 
   it "ignores empty error values" do