earthdaily / earthdaily/earthdaily-python-client

Add health check

Open
#212 4 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue
Dominant language
Python
Stars
31
Forks
10
PR merge metrics
No merged PRs in 30d

Description

I've apparently got invalid creds and am getting
```
requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://api.earthdaily.com
```
Ideally we would have a method to check credentials on
```python
# Initialize client
config = EDSConfig()
client = EDSClient(config)
```
It could raise a warning if the EDS_API_URL and EDS_AUTH_URL are correct but either of EDS_CLIENT_ID or secret are not

Contributor guide

Open the contributing guide

Research direction

Start at EDSClient(config) and trace the existing authentication request and error handling. Define how the health check should distinguish incorrect URLs from invalid client credentials, then add coverage for the reported 401 case and verify the resulting warning behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.