firefly GitHub
PR #796 | Backport fixes for 1.0.1 |
Backport the following PRs from main in preparation for a patch release: #775 #776 #777 #778 #780 #782 Also includes updates to the token connectors (only) in manifest.json. |
Created At 2022-05-06 16:09:47 +0000 UTC
PR #794 | Do not init apiserver config, until after config reset |
CORS defaults were not being applied any more, after #791 moved the initialization to be alongside all the rest of the HTTP server config. We were initializing those config prefixes, before we were calling `coreconfig.Reset()`  |
Created At 2022-05-05 21:22:01 +0000 UTC
PR #793 | Fix links in documentation |
I deleted the previous branch when i noticed my git credentials had not been updated in my signed commits. This PR still addresses the two links. |
Created At 2022-05-05 20:17:11 +0000 UTC
PR #792 | Allow namespace to be omitted from URLs |
Proposal to tweak our route generation so that namespaced routes will actually generate _two_ versions of the route: * a base route like `/data` that implies use of the default namespace * a namespaced route like `/namespaces/{ns}/data` that allows you to specify the namespace To reduce clutter on the Swagger UI page, routes will also now be grouped into 3 category buckets. I'm open to names for these categories, but I've proposed "Global", "Default Namespace", and "Non-Default Namespace". The main functional changes are in `internal/apiserver/routes.go`. --- ![]() ![]() ![]() ![]() |
Created At 2022-05-04 20:04:25 +0000 UTC
PR #791 | Refactor to move common utility code out to firefly-common |
- Moves base `pkg/fftypes` (time, numeric, bytes etc.) out to `firefly-common` - Moves all FireFly Core data types from `pkg/fftypes` to `pkg/core` - Removes the following packages that all moved to `firefly-common`: - `pkg/config` - `pkg/ffresty` - `pkg/httpserver` - `pkg/i18n` - `pkg/log` - `pkg/retry` - `pkg/wsclient` - Only functional change: `api.shutdownTimeout` moved to be consistent with the other `httpserver` config, so is now split into three separate configurations: - `api.shutdownTimeout` - `admin.shutdownTimeout` - `metrics.shutdownTimeout` - I decided to keep `cors.` as a global config shared across `api`/`admin`/`metrics` - which means initializing the HTTP Server now passes in two separate config prefixes |
Created At 2022-05-04 19:44:51 +0000 UTC
PR #790 | Split definition methods out of Broadcast Manager into new Definition Sender |
Not totally crazy about the names `defsender.Sender` and `defhandler.DefinitionHandler`, but they need to be separate packages and this is where I landed... |
Created At 2022-05-04 15:37:00 +0000 UTC
PR #789 | Remove non-definition items from DefinitionHandler |
Resolves #614 |
Created At 2022-05-03 18:49:31 +0000 UTC
PR #786 | update init logging branding |
Created At 2022-05-03 16:02:52 +0000 UTC </div> |