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.

Sign up for GitHub

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

Jump to bottom

Change Redact Mode Names #18

Closed
taylorb-syd opened this issue Feb 14, 2021 · 2 comments
Closed

Change Redact Mode Names #18

taylorb-syd opened this issue Feb 14, 2021 · 2 comments
Assignees
@taylorb-syd
Labels
enhancement
Milestone
1.2

Comments

@taylorb-syd
Copy link
Contributor

"Blacklist" and "Whitelist" have negative connotations, so need to change them to something more acceptable like "Allowlist" and "Blocklist".

Will need to include "Blacklist" and "Whitelist" in library, but unmentioned in documentation, for backwards compatibility. Probably best to put a redundancy message in the logs, and remove at a future date.
@taylorb-syd taylorb-syd self-assigned this Feb 14, 2021
@taylorb-syd taylorb-syd added this to the 2.0 milestone Feb 14, 2021
taylorb-syd added a commit that referenced this issue Feb 15, 2021
@taylorb-syd
[Issue #18] Changed Whitelist/Blacklist to Allowlist/Blocklist, Refac…
d10a476 
…tored Code, Improved Testing Code, Improved README.md grammar
@taylorb-syd
Copy link
Contributor Author

Ready to merge from issue-18 branch.

taylorb-syd added a commit that referenced this issue Feb 15, 2021
@taylorb-syd
[Issue #3 C2] Merge Conflict Resolution with Issue #18
407f301
@taylorb-syd
Copy link
Contributor Author

Merged into main. Closing out issue awaiting PR.

@taylorb-syd taylorb-syd mentioned this issue Feb 15, 2021
Merged
taylorb-syd added a commit that referenced this issue Feb 15, 2021
@taylorb-syd
[1.1.2] Changed Whitelist/Blacklist to Allowlist/Blocklist as per Issue
87521ea 
#18
taylorb-syd added a commit that referenced this issue Feb 15, 2021
@taylorb-syd
Revert "[1.1.2] Changed Whitelist/Blacklist to Allowlist/Blocklist as…
884ee9d 
… per Issue #18"

This reverts commit 87521ea.
taylorb-syd added a commit that referenced this issue Feb 15, 2021
@taylorb-syd
[1.1.2] Updated setup.py for 1.1.2 Cherry-Pick release of Issue #18
aa03bb9
@taylorb-syd taylorb-syd mentioned this issue Feb 15, 2021
Merged
taylorb-syd added a commit that referenced this issue Feb 15, 2021
@taylorb-syd
[1.1.2] Updated setup.py for 1.1.2 Cherry-Pick release of Issue #18
6134f3d
taylorb-syd added a commit that referenced this issue Feb 15, 2023
@taylorb-syd
[1.20] Completed Milestones for 1.2.0 release
04296e7 
* [Issue #18] Changed Whitelist/Blacklist to Allowlist/Blocklist, Refactored Code, Improved Testing Code, Improved README.md grammar

* [Issue #3] Introduced logic to check size of response. Refactored decorator exception handling.

* [Issue #4] Introduced a function to collapse the output data and used this in the cfnresponse function

* [1.2.0rc1] Documentation Pass to improve README.md

* [Issue #2] Implemented Unit Tests

* [Issue #21] Added the response headers for troubleshooting to the debug log output
taylorb-syd added a commit that referenced this issue Feb 15, 2023
@taylorb-syd
[1.2.0] Completed Milestones for 1.2.0 release
c416b40 
* [Issue #18] Changed Whitelist/Blacklist to Allowlist/Blocklist, Refactored Code, Improved Testing Code, Improved README.md grammar

* [Issue #3] Introduced logic to check size of response. Refactored decorator exception handling.

* [Issue #4] Introduced a function to collapse the output data and used this in the cfnresponse function

* [1.2.0rc1] Documentation Pass to improve README.md

* [Issue #2] Implemented Unit Tests

* [Issue #21] Added the response headers for troubleshooting to the debug log output
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@taylorb-syd taylorb-syd

Labels
enhancement
Projects
None yet
Milestone
1.2
Development

No branches or pull requests
1 participant
@taylorb-syd