solidus
solidus copied to clipboard
Remove blacklist and whitelist usage
For obvious reasons. Open to other naming suggestions of course.
There are other parts in the codebase that use blacklist, these are from 3rd party software that need to be updated when we can.
- [x] replace blacklist with deny
- [x] replace whitelist with allow
- [ ] track 3rd party software used and try to change it there as well.
- [ ] add deprecations for these methods
- [x] update documention and guides
Description
Checklist:
- [ ] I have followed Pull Request guidelines
- [ ] I have added a detailed description into each commit message
- [ ] I have updated Guides and README accordingly to this change (if needed)
- [ ] I have added tests to cover this change (if needed)
- [ ] I have attached screenshots to this PR for visual changes (if needed)
Hey @peterberkenbosch, having this work finished would be great. Is there anything else to do here? If not, could you please rebase from master and set it ready for review? :pray:
Closed with https://github.com/solidusio/solidus/pull/4853, thanks @RyanofWoods and @peterberkenbosch!