-
Notifications
You must be signed in to change notification settings - Fork 5
Fixes #11429: Update datasource to Rudder 4.2 license framework #32
Fixes #11429: Update datasource to Rudder 4.2 license framework #32
Conversation
val isEnabled = true | ||
val enabledStatus = DatasourcesStatus.Enabled | ||
val licenseInformation = None | ||
final class CheckRudderPluginDatasourcesEnableImpl() extends PluginStatus { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not using: AlwaysEnabledPluginStatus instead ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and removing this file (and removing lines from pom.xml if default is now useless
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because we need the correct name to be able to instanciate that service in the plugin module. I don't have any simple way to make that work without a full-blown IoC or Java Service arch. I don't want to use them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All right !!
|
||
maybeInfo.fold( error => DataSourceLogger.error(error) , ok => DataSourceLogger.warn("License signature is valid.") ) | ||
//log at that point is we read the license information for the plugin | ||
maybeInfo.fold( error => DataSourceLogger.error(error) , ok => DataSourceLogger.info("Plugin datasource has a license and the license signature is valid.") ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Datasource plugin no ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to write "Plugin '${plugin name}' actually :)
06fbc07
to
d09c18c
Compare
PR rebased |
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/11429