killbill-platform

killbill-platform copied to clipboard

With https://github.com/killbill/killbill-platform/issues/162, the log entries will contain the `accountRecordId` and `userToken`. Add support for filtering the stream by `userToken` or `accountId` (e.g., via a query parameter). The plugin will need...

pierre

In `KillbillLogWriter`, MDC entries can be fetched from `MDC.getCopyOfContextMap()`. Change `LogEntryJson` to contain these: `userToken`, `tenantRecordId`, `accountRecordId`.

pierre

SSE Logs - Error while closing the output stream in order to commit response

7

This is a follow up issue based on [silence SSE logs](https://github.com/killbill/killbill-platform/issues/61) issue. This problem showing up when: 1. `killbill` running with `kpm` and `logger` plugin installed. 2. SSE endpoint (`http://127.0.0.1/plugins/killbill-osgi-logger`)...

xsalefter

There's random exception thrown when installing plugin. ``` 2024-03-29T22:00:36,844+0000 lvl='ERROR', log='Bus', th='bus_events-th', xff='', rId='', tok='', aRId='', tRId='0', Exception thrown by subscriber method handleKillbillEvent(org.killbill.billing.notification.plugin.api.ExtBusEvent) on subscriber org.killbill.billing.osgi.OSGIListener@7cc8ae6c when dispatching event: DefaultBusExternalEvent{objectId=null,...

xsalefter

Download via kpm plugin fails if the plugin naming convention is different

1

The KPM plugin code tries to guess the artifactId by convention (PluginNamingResolver.of(pluginKey).getPluginName()). However, if the convention isn't respected, the download fails. If the caller has specified `pluginArtifactId`, the plugin should...

reshmabidikar

If [PluginNamingResolver](https://github.com/killbill/killbill-platform/blob/master/osgi-bundles/bundles/kpm/src/main/java/org/killbill/billing/osgi/bundles/kpm/impl/PluginNamingResolver.java#L64) created without specifying the `pluginVersion`, [getPluginVersion() throw IllegalStateException](https://github.com/killbill/killbill-platform/blob/e794cf8ca7508e46d3cf8bf5b09df2edbba17fa8/osgi-bundles/bundles/kpm/src/main/java/org/killbill/billing/osgi/bundles/kpm/impl/PluginNamingResolver.java#L63). It is not correct, since we have a chances to analyzing version from `stringContainsVersion`. IIRC, that [assertion](https://github.com/killbill/killbill-platform/blob/e794cf8ca7508e46d3cf8bf5b09df2edbba17fa8/osgi-bundles/bundles/kpm/src/main/java/org/killbill/billing/osgi/bundles/kpm/impl/PluginNamingResolver.java#L64) added when working...

xsalefter

In `KillbillActivator`, register the `HealthCheckRegistry` as an OSGI service, so that plugins can access other plugins and Kill Bill healthchecks. https://github.com/killbill/killbill-platform/blob/4fa797eaa30c82d0821c85daddf4a2e647794379/osgi/src/main/java/org/killbill/billing/osgi/KillbillActivator.java#L205

pierre

De-register healthchecks/metrics, when the stop() method is called on the bundle/plugin.

vnandwana

InfluxDB writer plugin: https://github.com/iZettle/dropwizard-metrics-influxdb Supports authString when sending metrics to influxdb. Counterpart in killbill doesn't support it, thus authenticated access is not supported.

artur-fijalkowski

Set thread contextClassLoader to the one from the bundle when activating a bundle

2

See https://github.com/killbill/killbill-platform/issues/5.

pierre