Skip to content
This repository has been archived by the owner on Aug 23, 2019. It is now read-only.

refactor: rename blacklist to denylist #341

Merged
merged 1 commit into from
Aug 1, 2019
Merged

Conversation

alanshaw
Copy link
Member

@alanshaw alanshaw commented Aug 1, 2019

"denylist" more adequately describes what this list is for.

BREAKING CHANGE: Constructor options blacklistTTL and blackListAttempts have been renamed to denyTTL and denyAttempts. The error code from errors thrown when dial is currently denied has changed from ERR_BLACKLISTED to ERR_DENIED.

"denylist" more adequately describes what this list is for.

BREAKING CHANGE: Constructor options `blacklistTTL` and `blackListAttempts` have been renamed to `denyTTL` and `denyAttempts`. The error code from errors thrown when dial is currently denied has changed from `ERR_BLACKLISTED` to `ERR_DENIED`.

License: MIT
Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
@alanshaw alanshaw requested a review from jacobheun August 1, 2019 10:55
@alanshaw alanshaw changed the title refactor: blacklist to denylist refactor: rename blacklist to denylist Aug 1, 2019
Copy link
Contributor

@jacobheun jacobheun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you!

@jacobheun jacobheun merged commit f64c73b into master Aug 1, 2019
@jacobheun jacobheun deleted the refactor/denylist branch August 1, 2019 11:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants