FIX: 2N+1 sql in admin /api/keys endpoint (#14679)

This commit is contained in:
Jarek Radosz
2021-10-21 17:00:17 +02:00
committed by GitHub
parent db53c6b248
commit 3b90d7de66

View File

@@ -5,7 +5,9 @@ class Admin::ApiController < Admin::AdminController
# If we used "api_key", then our user provider would try to use the value for authentication # If we used "api_key", then our user provider would try to use the value for authentication
def index def index
keys = ApiKey.where(hidden: false) keys = ApiKey
.where(hidden: false)
.includes(:user, :api_key_scopes)
# Put active keys first # Put active keys first
# Sort active keys by created_at, sort revoked keys by revoked_at # Sort active keys by created_at, sort revoked keys by revoked_at