Skip to content

Commit

Permalink
Remove Octokit::Client::Authorizations (#1494)
Browse files Browse the repository at this point in the history
* Remove all but scopes and create_authorization methods

* Rename to Tokens module

* Remove create_authorization in favor of hardcoded test tokens

* Remove unnecessary cassettes

* Fix comment
  • Loading branch information
kfcampbell authored Oct 20, 2022
1 parent 8413882 commit 017b0e3
Show file tree
Hide file tree
Showing 18 changed files with 77 additions and 449 deletions.
2 changes: 1 addition & 1 deletion .rubocop_todo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@ Style/IfUnlessModifier:
Exclude:
- 'lib/octokit.rb'
- 'lib/octokit/client.rb'
- 'lib/octokit/client/authorizations.rb'
- 'lib/octokit/client/tokens.rb'
- 'lib/octokit/client/commits.rb'
- 'lib/octokit/client/organizations.rb'
- 'lib/octokit/client/repositories.rb'
Expand Down
4 changes: 2 additions & 2 deletions lib/octokit/client.rb
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@
require 'octokit/client/actions_workflow_jobs'
require 'octokit/client/actions_workflow_runs'
require 'octokit/client/apps'
require 'octokit/client/authorizations'
require 'octokit/client/checks'
require 'octokit/client/commits'
require 'octokit/client/commit_comments'
Expand Down Expand Up @@ -62,6 +61,7 @@
require 'octokit/client/source_import'
require 'octokit/client/stats'
require 'octokit/client/statuses'
require 'octokit/client/tokens'
require 'octokit/client/traffic'
require 'octokit/client/users'
require 'ext/sawyer/relation'
Expand All @@ -77,7 +77,6 @@ class Client
include Octokit::Warnable
include Octokit::Client::ActionsArtifacts
include Octokit::Client::ActionsSecrets
include Octokit::Client::Authorizations
include Octokit::Client::Checks
include Octokit::Client::Commits
include Octokit::Client::CommitComments
Expand Down Expand Up @@ -126,6 +125,7 @@ class Client
include Octokit::Client::SourceImport
include Octokit::Client::Stats
include Octokit::Client::Statuses
include Octokit::Client::Tokens
include Octokit::Client::Traffic
include Octokit::Client::Users

Expand Down
185 changes: 0 additions & 185 deletions lib/octokit/client/authorizations.rb

This file was deleted.

31 changes: 31 additions & 0 deletions lib/octokit/client/tokens.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
# frozen_string_literal: true

module Octokit
class Client
# Method to check scopes
#
# @see https://developer.github.com/v3/oauth_authorizations/#oauth-authorizations-api
module Tokens
# Check scopes for a token
#
# @param token [String] GitHub OAuth token
# @param options [Hash] Header params for request
# @return [Array<String>] OAuth scopes
# @see https://developer.github.com/v3/oauth/#scopes
def scopes(token = @access_token, options = {})
options = options.dup
raise ArgumentError, 'Access token required' if token.nil?

auth = { 'Authorization' => "token #{token}" }
headers = (options.delete(:headers) || {}).merge(auth)

agent.call(:get, 'user', headers: headers)
.headers['X-OAuth-Scopes']
.to_s
.split(',')
.map(&:strip)
.sort
end
end
end
end

This file was deleted.

This file was deleted.

This file was deleted.

Loading

0 comments on commit 017b0e3

Please sign in to comment.