7dc0eba48a
The proxy method adds an extra slash even if the route param has the character. It’s challenging to check the expected format on each route. The simplest solution is to check if the route param begins with a slash, and if not, append one. NB: The existing tests are sufficient to cover this case. There’s no need for an additional test to specifically test this.
54 lines
1.4 KiB
Ruby
54 lines
1.4 KiB
Ruby
class Api::V1::Accounts::Integrations::CaptainController < Api::V1::Accounts::BaseController
|
|
before_action :hook
|
|
|
|
def proxy
|
|
response = HTTParty.send(request_method, request_url, body: permitted_params[:body].to_json, headers: headers)
|
|
render plain: response.body, status: response.code
|
|
end
|
|
|
|
private
|
|
|
|
def headers
|
|
{
|
|
'X-User-Email' => hook.settings['account_email'],
|
|
'X-User-Token' => hook.settings['access_token'],
|
|
'Content-Type' => 'application/json',
|
|
'Accept' => '*/*'
|
|
}
|
|
end
|
|
|
|
def request_path
|
|
request_route = with_leading_hash_on_route(params[:route])
|
|
|
|
return 'api/sessions/profile' if request_route == '/sessions/profile'
|
|
|
|
"api/accounts/#{hook.settings['account_id']}#{request_route}"
|
|
end
|
|
|
|
def request_url
|
|
base_url = InstallationConfig.find_by(name: 'CAPTAIN_API_URL').value
|
|
URI.join(base_url, request_path).to_s
|
|
end
|
|
|
|
def hook
|
|
@hook ||= Current.account.hooks.find_by!(app_id: 'captain')
|
|
end
|
|
|
|
def request_method
|
|
method = permitted_params[:method].downcase
|
|
raise 'Invalid or missing HTTP method' unless %w[get post put patch delete options head].include?(method)
|
|
|
|
method
|
|
end
|
|
|
|
def with_leading_hash_on_route(request_route)
|
|
return '' if request_route.blank?
|
|
|
|
request_route.start_with?('/') ? request_route : "/#{request_route}"
|
|
end
|
|
|
|
def permitted_params
|
|
params.permit(:method, :route, body: {})
|
|
end
|
|
end
|