GitLab

Gitlab ci testing

See merge request !22

Resolve "Use ThreadPoolExecutor in place of ProcessPoolExecutor"

Closes #40

See merge request !20

We will need to test how this scales with added clients

Each client starts 2 threads for the publish and
broadcast flow.
In addition there are 2 threads for the publisher
and config watcher.

Resolve "Run subscribes on main thread"

Closes #39

See merge request !19

Resolve "Fix log write in rmq subscribe"

Closes #38

See merge request !18

Resolve "Add DEBUG log writes to soar_bus to show messages in transit"

Closes #35

See merge request !16

- set log format
- remove or replace prints with log writes
- change log levels for some messages

Log failed authentication
Log failed token grants

Set global log config for bus and api
Add TRACE level
Log actions for DEBUG
Log full messages for TRACE

LOG_LEVEL is passed into docker-compose with default DEBUG
For production it defaults to INFO if not set

Log an event when messages are:

- written to queues
- read from queues
- forwarded
- published
- broadcast

fix: correct error code resp

Closes #32 and #34

See merge request !17

refactor: better err msgs for invalid JSON

Resolve "Make API CORS settings configurable in config settings"

Closes #28

See merge request !14

Default settings are applied if the config file is not present.
The default settings assume running a local dev env.
Config should always be in place for production.

Resolve "Fix error running without any clients defined"

Closes #31

See merge request !15

+ blacken changed files
+ add client_create.py to readme
+ don't create admin client automatically

Creating a default client the user hasn't asked for could
create problems with unused clients building up big queues

The code was catching a FileNotFoundError and trying
to create the empty file but wasn't creating the
data directory before trying to save into it.

Rather than starting with 0 clients it now creates
a default "admin" client so that when you start the
app you always have something.

We need to think about how to provide these credentials
back to the user. Normally you save the response from
the post to /client. You can just read the file.

The admin client is set to receive all messages.
It may be better to set it to receive nothing so that
if the default client is not implemented the queues don't
fill up.

Resolve "Detect new clients and add to thread pool"

Closes #27

See merge request !13