mirror of
https://github.com/GSA/notifications-admin.git
synced 2026-02-05 10:53:28 -05:00
Don’t set combined API on Notify python client
Because we’re setting the API key and service ID after calling the `__init__` method of the client it wasn’t doing the thing where it splits the combined key into the two individual UUIDs. So we still need to set them directly, individually on the client.
This commit is contained in:
@@ -8,10 +8,8 @@ class StatusApiClient(NotifyAdminAPIClient):
|
||||
|
||||
def init_app(self, app):
|
||||
self.base_url = app.config['API_HOST_NAME']
|
||||
self.api_key = '{}-{}'.format(
|
||||
app.config['ADMIN_CLIENT_USER_NAME'],
|
||||
app.config['ADMIN_CLIENT_SECRET'],
|
||||
)
|
||||
self.service_id = app.config['ADMIN_CLIENT_USER_NAME']
|
||||
self.api_key = app.config['ADMIN_CLIENT_SECRET']
|
||||
|
||||
def get_status(self, *params):
|
||||
return self.get(url='/_status', *params)
|
||||
|
||||
Reference in New Issue
Block a user