Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduction of the Enabled-Switch for Collectors. #718

Merged
merged 1 commit into from
Nov 8, 2016

Conversation

dmth
Copy link
Contributor

@dmth dmth commented Oct 4, 2016

This introduces the enabled-switch for all Collector-Bots.
By doing so it fixes #715.

The Collectorbot was extended with an own start-function
which checks if the enabled parameter is present and was set
to a representation of False. To check the "representation of
False" the function to_bool was introduced in utils.py, which
maps well-known-text to booleans. It has a a fallbackmode parameter
which determines if the function returns true of false if no
mapping was found.

As long as to_bool does not evaluate to False the bot will be started.

This introduces the enabled-switch for all Collector-Bots.
By doing so it fixes certtools#715.

The Collectorbot was extended with an own start-function
which checks if the enabled paramter is present and was set
to a representation of False. To check the "representation of
False" the function to_bool was introduced in utils.py, which
maps well-known-text to booleans. It has a a fallbackmode parameter
which determines if the function returns true of false if no
mapping  was found.

As long as to_bool does not evaluate to False the bot will be started.
@sebix sebix added this to the v1.0 Stable Release milestone Oct 4, 2016
@sebix sebix added feature Indicates new feature requests or new features needs: review component: core labels Oct 4, 2016
@sebix sebix self-assigned this Oct 4, 2016
@sebix sebix merged commit 9d0db8a into certtools:master Nov 8, 2016
@sebix
Copy link
Member

sebix commented Nov 8, 2016

I pushed a generalized version to master.

@dmth dmth deleted the dev-collectorbot-enabled-switch branch November 8, 2016 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: core feature Indicates new feature requests or new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants