From ade5a8eed918de46b76da085efe78859409026e0 Mon Sep 17 00:00:00 2001
From: Ken Mayer <ken.mayer@meraki.net>
Date: Fri, 26 Jul 2024 17:47:37 -0700
Subject: [PATCH] Revert "DRY out AuthorizationsController#render_success"

This reverts commit f0d48d6f4ca03f14a987dbe8baa1269acd3784c8.
---
 CHANGELOG.md                                            | 1 -
 app/controllers/doorkeeper/authorizations_controller.rb | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 683d39a50..edc7c81b5 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -8,7 +8,6 @@ User-visible changes worth mentioning.
 ## main
 
 Add your entry here.
-- [#????] DRY out AuthorizationsController#render_success
 
 - [#1715] Fix token introspection invalid request reason
 - [#1714] Fix `Doorkeeper::AccessToken.find_or_create_for` with empty scopes which raises NoMethodError
diff --git a/app/controllers/doorkeeper/authorizations_controller.rb b/app/controllers/doorkeeper/authorizations_controller.rb
index 97dc1adaa..9452167f7 100644
--- a/app/controllers/doorkeeper/authorizations_controller.rb
+++ b/app/controllers/doorkeeper/authorizations_controller.rb
@@ -32,7 +32,7 @@ def destroy
 
     def render_success
       if skip_authorization? || can_authorize_response?
-        create
+        redirect_or_render(authorize_response)
       elsif Doorkeeper.configuration.api_only
         render json: pre_auth
       else