Remove `trigger_request.trigger.owner` from app/services/ci/create_pipeline_service.rb
The following discussion from !13102 (merged) should be addressed:
-
@godfat commented on a discussion: (+9 comments) Sorry for raising this again, but I want to comment on this line.
Since we no longer pass
trigger_request
here, we need to make sure thatcurrent_user
must exist otherwise this would raise an error.I am unfamiliar with API authentication, could that be possible? If it's impossible, we could just remove
|| trigger_request.trigger.owner
. However, that would also break other assumptions. For example, the part that:# legacy triggers don't have a corresponding user
would no longer work.So I "guess"
current_user
could benil
?