-
Notifications
You must be signed in to change notification settings - Fork 624
Replace whitelist/blacklist with safe/block-list #2532
Copy link
Copy link
Closed
Labels
good first issueGood for newcomersGood for newcomers
Description
Lagom uses the terms whitelist/blackslit in a couple of places (internally and externally).
It'd be nice to adopt other naming (see for example TravisCI moving to safelist/blocklist).
There's at least the exception-whitelist:
lagom/service/core/client/src/main/resources/reference.conf
Lines 24 to 27 in d021b2a
| # A whitelist of fqcn of Exceptions that the CircuitBreaker | |
| # should not consider failures. By default all exceptions are | |
| # considered failures. | |
| exception-whitelist = [] |
And the dependencyWhitelist:
lagom/project/Dependencies.scala
Line 133 in d021b2a
| val DependencyWhitelist: Def.Initialize[Seq[ModuleID]] = Def.setting { |
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomers