Skip to content

Commit

Permalink
Changed scope of BASE_URL
Browse files Browse the repository at this point in the history
  • Loading branch information
sferik committed Nov 18, 2017
1 parent 4e41d0c commit b40e208
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/t/cli.rb
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ def authorize
open_or_print('https://apps.twitter.com', dry_run: options['display-uri'])
key = ask 'Enter your API key:'
secret = ask 'Enter your API secret:'
consumer = OAuth::Consumer.new(key, secret, site: Twitter::REST::Client::BASE_URL)
consumer = OAuth::Consumer.new(key, secret, site: Twitter::REST::Request::BASE_URL)
request_token = consumer.get_request_token
uri = generate_authorize_uri(consumer, request_token)
say
Expand Down Expand Up @@ -973,7 +973,7 @@ def generate_authorize_uri(consumer, request_token)
value =~ /"(.*?)"/
"#{key}=#{CGI.escape(Regexp.last_match[1])}"
end.join('&')
"#{Twitter::REST::Client::BASE_URL}#{request.path}?#{params}"
"#{Twitter::REST::Request::BASE_URL}#{request.path}?#{params}"
end

def pin_auth_parameters
Expand Down

2 comments on commit b40e208

@jpf
Copy link

@jpf jpf commented on b40e208 Jan 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! This patch allowed me to get t working on my machine.

@ilkermutlu
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When should we expect a new gem release including this patch?

Please sign in to comment.