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

Add Masked Fields Support #395

Closed
desertfoxmb opened this issue Mar 1, 2018 · 3 comments
Closed

Add Masked Fields Support #395

desertfoxmb opened this issue Mar 1, 2018 · 3 comments
Assignees
Labels
discussion enhancement An improvement or new feature V6

Comments

@desertfoxmb
Copy link

Q A
Bug? no
New Feature? yes
Framework Laravel
Framework version 5.5
Package version 6.0.1
PHP version 7.0.1

Actual Behaviour

No field masking allowed

Desired Behaviour

Add the ability to have a masked fields property. The desired behavior would be to still include the field in the audit (unless it was in the exclude list) but to mask it. Additionally it may be good to have a configuration for "MaskAllButLastXChars" which would take the number of characters to leave exposed.

For example, with something like a social security or credit card number, supposing MaskAlLButLastXChars = 4, replace the new/old values with ***1234. It would still be clear that the value was in the change set but information security wouldn't be violated.

@quetzyg quetzyg self-assigned this Mar 4, 2018
@quetzyg quetzyg added enhancement An improvement or new feature discussion V6 labels Mar 4, 2018
@quetzyg
Copy link
Contributor

quetzyg commented Mar 4, 2018

Hi @desertfoxmb, sorry I haven't replied sooner.

I like your idea, it's an awesome suggestion.
Let me see if I can implement it during this month.

I'll leave this ticket open.
Thank you!

@desertfoxmb
Copy link
Author

No worries, thanks for getting back when you did. I'm glad you think that idea is useful. Great job on the project. Easy to implement and easy to customize data that's saved if necessary.

This was referenced Apr 6, 2018
@quetzyg
Copy link
Contributor

quetzyg commented Apr 9, 2018

This feature is now available in master. Will tag soon, once the documentation is updated.

@quetzyg quetzyg closed this as completed Apr 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion enhancement An improvement or new feature V6
Projects
None yet
Development

No branches or pull requests

2 participants