Commit 7d7cd4ea authored by Sean McGivern's avatar Sean McGivern

Add feature categories to controllers beginning with O

parent bddf5186
...@@ -8,6 +8,8 @@ class Oauth::Jira::AuthorizationsController < ApplicationController ...@@ -8,6 +8,8 @@ class Oauth::Jira::AuthorizationsController < ApplicationController
skip_before_action :authenticate_user! skip_before_action :authenticate_user!
skip_before_action :verify_authenticity_token skip_before_action :verify_authenticity_token
feature_category :integrations
# 1. Rewire Jira OAuth initial request to our stablished OAuth authorization URL. # 1. Rewire Jira OAuth initial request to our stablished OAuth authorization URL.
def new def new
session[:redirect_uri] = params['redirect_uri'] session[:redirect_uri] = params['redirect_uri']
......
...@@ -11,6 +11,8 @@ class OmniauthCallbacksController < Devise::OmniauthCallbacksController ...@@ -11,6 +11,8 @@ class OmniauthCallbacksController < Devise::OmniauthCallbacksController
protect_from_forgery except: [:kerberos, :saml, :cas3, :failure], with: :exception, prepend: true protect_from_forgery except: [:kerberos, :saml, :cas3, :failure], with: :exception, prepend: true
feature_category :authentication_and_authorization
def handle_omniauth def handle_omniauth
omniauth_flow(Gitlab::Auth::OAuth) omniauth_flow(Gitlab::Auth::OAuth)
end end
......
...@@ -5,6 +5,8 @@ class OmniauthKerberosSpnegoController < ApplicationController ...@@ -5,6 +5,8 @@ class OmniauthKerberosSpnegoController < ApplicationController
skip_before_action :authenticate_user! skip_before_action :authenticate_user!
feature_category :authentication_and_authorization
def negotiate def negotiate
if spnego_provided? && (krb_principal = spnego_credentials!(spnego_token)) if spnego_provided? && (krb_principal = spnego_credentials!(spnego_token))
session[OmniAuth::Strategies::KerberosSpnego::SESSION_KEY] = krb_principal session[OmniAuth::Strategies::KerberosSpnego::SESSION_KEY] = krb_principal
......
...@@ -5,6 +5,8 @@ class OperationsController < ApplicationController ...@@ -5,6 +5,8 @@ class OperationsController < ApplicationController
respond_to :json, only: [:list] respond_to :json, only: [:list]
feature_category :release_orchestration
POLLING_INTERVAL = 120_000 POLLING_INTERVAL = 120_000
def index def index
......
...@@ -27,7 +27,7 @@ RSpec.describe "Every controller" do ...@@ -27,7 +27,7 @@ RSpec.describe "Every controller" do
next unless controller.to_s.start_with?('B', 'C', 'D', 'E', 'F', next unless controller.to_s.start_with?('B', 'C', 'D', 'E', 'F',
'H', 'I', 'J', 'K', 'L', 'H', 'I', 'J', 'K', 'L',
'M', 'N', 'M', 'N', 'O',
'Projects::MergeRequestsController') 'Projects::MergeRequestsController')
"#{controller}##{action}" "#{controller}##{action}"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment