Skip to content

Commit

Permalink
Added missing import for 'os' module
Browse files Browse the repository at this point in the history
Was causing an exception leading to "push not configured" message everytime irrespective of whether config files present or env variables configured.

Code with error:

```
try:
logging.info(f"Push configured for app {push_config.get('PUSH_SERVER_AUTH_TOKEN')} using platform {os.getenv('PUSH_PROVIDER')} with token {os.getenv('PUSH_SERVER_AUTH_TOKEN')[:10]}... of length {len(os.getenv('PUSH_SERVER_AUTH_TOKEN'))}")
except:
logging.warning("push service not configured, push notifications not supported")
```
  • Loading branch information
Mahadik, Mukul Chandrakant authored and Mahadik, Mukul Chandrakant committed Sep 4, 2024
1 parent b2aefce commit 6368598
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions emission/net/ext_service/push/notify_interface.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
import json
import logging
import importlib
import os

import emission.core.backwards_compat_config as ecbc

Expand Down

0 comments on commit 6368598

Please sign in to comment.