Skip to content

Commit 9f811fb

Browse files
author
Unai Abrisketa
authored
Fix access token requirement and fix logger (#18)
1 parent c5a442d commit 9f811fb

File tree

6 files changed

+16
-9
lines changed

6 files changed

+16
-9
lines changed

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,9 @@
1+
### v0.7.1.pre
2+
3+
* bug-fixes
4+
* Allow to create a `BeyondApi::Session` without `access_token` and `refresh_token`
5+
* Fix no-Rails logger
6+
17
### v0.7.0.pre
28

39
* features

Gemfile.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
PATH
22
remote: .
33
specs:
4-
beyond_api (0.7.0.pre)
4+
beyond_api (0.7.1.pre)
55
faraday (~> 0.15)
66

77
GEM

lib/beyond_api.rb

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@
77
require "beyond_api/ext"
88
require "beyond_api/utils"
99

10+
require "logger"
11+
1012
module BeyondApi
1113
class Error < StandardError; end
1214

lib/beyond_api/request.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ class << self
99
define_method(method) do |session, path, params = {}|
1010
response = BeyondApi::Connection.default.send(method) do |request|
1111
request.url(session.api_url + path)
12-
request.headers['Authorization'] = "Bearer #{ session.access_token }"
12+
request.headers['Authorization'] = "Bearer #{ session.access_token }" unless session.access_token.nil?
1313
request.params = params.to_h.camelize_keys
1414
end
1515

@@ -21,7 +21,7 @@ class << self
2121
define_method(method) do |session, path, body = {}, params = {}|
2222
response = BeyondApi::Connection.default.send(method) do |request|
2323
request.url(session.api_url + path)
24-
request.headers['Authorization'] = "Bearer #{ session.access_token }"
24+
request.headers['Authorization'] = "Bearer #{ session.access_token }" unless session.access_token.nil?
2525
request.params = params.to_h.camelize_keys
2626
request.body = body.camelize_keys.to_json
2727
end
@@ -34,7 +34,7 @@ class << self
3434
def self.upload(session, path, file_binary, content_type, params)
3535
response = BeyondApi::Connection.default.post do |request|
3636
request.url(session.api_url + path)
37-
request.headers['Authorization'] = "Bearer #{ session.access_token }"
37+
request.headers['Authorization'] = "Bearer #{ session.access_token }" unless session.access_token.nil?
3838
request.headers['Content-Type'] = content_type
3939
request.params = params.to_h.camelize_keys
4040
request.body = file_binary

lib/beyond_api/resources/base.rb

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,9 @@ class InvalidSessionError < StandardError; end
88

99
def initialize(session)
1010
@session = session
11-
raise InvalidSessionError.new("Invalid session") unless session.is_a? BeyondApi::Session
12-
if session.api_url.nil? || session.access_token.nil? || session.refresh_token.nil?
13-
raise InvalidSessionError.new("Session api_url, access_token and refresh_token cannot be nil")
14-
end
11+
12+
raise InvalidSessionError, "Invalid session" unless session.is_a? BeyondApi::Session
13+
raise InvalidSessionError, "Session api_url cannot be nil" if session.api_url.nil?
1514
end
1615
end
1716
end

lib/beyond_api/version.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
module BeyondApi
2-
VERSION = "0.7.0.pre".freeze
2+
VERSION = "0.7.1.pre".freeze
33
end

0 commit comments

Comments
 (0)