Skip to content

Commit 0165723

Browse files
SK-1731: Fix api key inconsistency (#149)
1 parent 83d4188 commit 0165723

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

skyflow/utils/validations/_validations.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import json
2-
import re
32
from skyflow.service_account import is_expired
43
from skyflow.utils.enums import LogLevel, Env, RedactionType, TokenMode
54
from skyflow.error import SkyflowError
@@ -45,12 +44,15 @@ def validate_required_field(logger, config, field_name, expected_type, empty_err
4544
raise SkyflowError(empty_error, invalid_input_error_code)
4645

4746
def validate_api_key(api_key: str, logger = None) -> bool:
47+
if not api_key.startswith('sky-'):
48+
log_error_log(SkyflowMessages.ErrorLogs.INVALID_API_KEY.value, logger=logger)
49+
return False
50+
4851
if len(api_key) != 42:
4952
log_error_log(SkyflowMessages.ErrorLogs.INVALID_API_KEY.value, logger = logger)
5053
return False
51-
api_key_pattern = re.compile(r'^sky-[a-zA-Z0-9]{5}-[a-fA-F0-9]{32}$')
5254

53-
return bool(api_key_pattern.match(api_key))
55+
return True
5456

5557
def validate_credentials(logger, credentials, config_id_type=None, config_id=None):
5658
key_present = [k for k in ["path", "token", "credentials_string", "api_key"] if credentials.get(k)]

0 commit comments

Comments
 (0)