Skip to content

Commit 1f962e1

Browse files
committed
[BaseController::Logger] Use miq_user_role.name
While `virtual_delegates` can be used to improve performance via a single attribute in a query, we already bring back the `miq_user_role` (via the lookup_scope), so it is better to just use that. And currently virtual_delegates don't support already loaded relationships.
1 parent fcdc4ec commit 1f962e1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

app/controllers/api/base_controller/logger.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ def log_api_auth
2323
group = User.current_user.current_group
2424
log_request("Authorization", :user => User.current_user.userid,
2525
:group => group.description,
26-
:role => group.miq_user_role_name,
26+
:role => group.miq_user_role.name,
2727
:tenant => group.tenant.name)
2828
end
2929
end

0 commit comments

Comments
 (0)