Skip to content

Commit

Permalink
🐛 HeaderClient: no "config"
Browse files Browse the repository at this point in the history
In debugging/development, we may use the "HeaderClient" instead of the
real "ConfidantClient". Some strange behaviour here made my debugging
journey more difficult, so lets fix that.

in `models.py:32`
```
   from secretupdater.headerclient import HeaderClient as ConfidantClient
```

`models.py:182`
```
    client = ConfidantClient(
      ...
    )
    app.logger.debug(client.config)
```

this fails because `client` (when its `HeaderClient`) doesnt have a
`.config`. Lets just give it one.
  • Loading branch information
matthope committed Feb 14, 2024
1 parent b5ba21c commit e05aeba
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions secretupdater/secretupdater/headerclient.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ class HeaderClient():
def __init__(self, **_kwargs):
super(HeaderClient, self).__init__()
app.logger.debug("Initialising HeaderClient")
self.config = {}

def get_service(self, service):
app.logger.debug("DummyClient.get_service")
Expand Down

0 comments on commit e05aeba

Please sign in to comment.