fix some styling offender

This commit is contained in:
Lin Jen-Shin 2016-05-18 17:43:00 -05:00
parent c337e748d3
commit e7a6d17b2b
3 changed files with 5 additions and 5 deletions

View File

@ -97,7 +97,7 @@ class OmniauthCallbacksController < Devise::OmniauthCallbacksController
handle_signup_error
end
def handle_service_ticket provider, ticket
def handle_service_ticket(provider, ticket)
Gitlab::OAuth::Session.create provider, ticket
session[:service_tickets] ||= {}
session[:service_tickets][provider] = ticket

View File

@ -285,8 +285,8 @@ module Ci
project.runners_token
end
def valid_token? token
project.valid_runners_token? token
def valid_token?(token)
project.valid_runners_token?(token)
end
def can_be_served?(runner)

View File

@ -947,13 +947,13 @@ class Project < ActiveRecord::Base
shared_runners_enabled? && Ci::Runner.shared.active.any?(&block)
end
def valid_runners_token? token
def valid_runners_token?(token)
self.runners_token && ActiveSupport::SecurityUtils.variable_size_secure_compare(token, self.runners_token)
end
# TODO (ayufan): For now we use runners_token (backward compatibility)
# In 8.4 every build will have its own individual token valid for time of build
def valid_build_token? token
def valid_build_token?(token)
self.builds_enabled? && self.runners_token && ActiveSupport::SecurityUtils.variable_size_secure_compare(token, self.runners_token)
end