Skip to content

Commit 3165b55

Browse files
authored
fix: improve OCS signing error messages (#189)
Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
1 parent 5238fda commit 3165b55

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

context_chat_backend/ocs_utils.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,13 @@ def _sign_request(headers: dict, username: str = '') -> None:
2525
# We assume that the env variables are set
2626
def _verify_signature(headers: Headers) -> str | None:
2727
if headers.get('EX-APP-ID') is None or headers.get('EX-APP-ID') != getenv('APP_ID'):
28-
logger.error(f'Invalid EX-APP-ID:{headers.get("EX-APP-ID")} != {getenv("APP_ID")}')
28+
logger.error(f'Invalid EX-APP-ID received: "{headers.get("EX-APP-ID")}", expected "{getenv("APP_ID")}"')
2929
return None
3030

3131
if headers.get('EX-APP-VERSION') is None or headers.get('EX-APP-VERSION') != getenv('APP_VERSION'):
3232
logger.error(
33-
f'Invalid EX-APP-VERSION:{headers.get("EX-APP-VERSION")} <=> {getenv("APP_VERSION")}'
33+
f'Invalid EX-APP-VERSION received: "{headers.get("EX-APP-VERSION")}", expected "{getenv("APP_VERSION")}".'
34+
' A reinstall of the app context_chat_backend in app_api keeping the data can potentially fix it.'
3435
)
3536
return None
3637

@@ -42,7 +43,7 @@ def _verify_signature(headers: Headers) -> str | None:
4243
username, app_secret = auth_aa.split(':', maxsplit=1)
4344

4445
if app_secret != getenv('APP_SECRET'):
45-
logger.error(f'Invalid APP_SECRET:{app_secret} != {getenv("APP_SECRET")}')
46+
logger.error('Invalid APP_SECRET received')
4647
return None
4748

4849
return username

0 commit comments

Comments
 (0)