Skip to content

Commit b7c9881

Browse files
committed
Raise missing jwt token error in session_id_from_shopify_id_token
1 parent 3ed00c4 commit b7c9881

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

lib/shopify_api/utils/session_utils.rb

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,13 @@ def current_session_id(auth_header, cookies, online)
4343

4444
sig do
4545
params(
46-
id_token: String,
46+
id_token: T.nilable(String),
4747
online: T::Boolean,
4848
).returns(String)
4949
end
5050
def session_id_from_shopify_id_token(id_token:, online:)
51+
raise Errors::MissingJwtTokenError, "Missing Shopify ID Token" unless id_token
52+
5153
payload = Auth::JwtPayload.new(id_token)
5254
shop = payload.shop
5355

test/utils/session_utils_test.rb

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,14 @@ def test_session_id_from_shopify_id_token_raises_invalid_jwt_errors
4949
end
5050
end
5151

52+
def test_session_id_from_shopify_id_token_raises_missing_jwt_token_error
53+
error = assert_raises(ShopifyAPI::Errors::MissingJwtTokenError) do
54+
ShopifyAPI::Utils::SessionUtils.session_id_from_shopify_id_token(id_token: nil, online: true)
55+
end
56+
57+
assert_equal("Missing Shopify ID Token", error.message)
58+
end
59+
5260
def test_non_embedded_app_current_session_id_raises_cookie_not_found_error
5361
ShopifyAPI::Context.stubs(:embedded?).returns(false)
5462

0 commit comments

Comments
 (0)