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

feat: enable multimasking function #20

Merged
merged 7 commits into from
May 30, 2024

Conversation

JEZAPAT
Copy link
Contributor

@JEZAPAT JEZAPAT commented May 28, 2024

Se requiere evolucionar la capacidad de enmascaramiento de datos que hoy en día se ofrece con el objetivo de que sea dinámica.

Actualmente, solo se permite enmascarar parte del string al principio, al final o en medio, no por palabra como lo requiere el negocio.

Ejemplo de como funciona actualmente: "Seb*** ****** *****"

Ejemplo de como se espera el nuevo funcionamiento: "Seb**** Cue*** Nav***"

Se agregaron dos propiedades opcionales al enmascaramiento las cuales habilitan el enmascaramiento múltiple generando el resultado esperado, las cuales son isMultiMask y separator.
isMultiMask es una bandera con la que se habilita la funcionalidad
separator es un string con el cual se puede definir el carácter por el cual se va a separar el campo ingresado.

Al momento de dividir el campo a enmascarar por medio del separator se reutiliza la funcionalidad que existía para no afectar el comportamiento actual del código.

Se hicieron pruebas implementando la libreria en el Distribution Router.

Copy link
Contributor

@gabheadz gabheadz left a comment

Choose a reason for hiding this comment

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

Please use conventional commits https://www.conventionalcommits.org/en/v1.0.0/ to describe changes made in the commits.

@JEZAPAT JEZAPAT changed the title Evolucionar la capacidad de enmascaramiento de datos Feature: enable multimasking function May 28, 2024
@JEZAPAT JEZAPAT changed the title Feature: enable multimasking function feat: enable multimasking function May 28, 2024
JEZAPAT and others added 7 commits May 28, 2024 14:40
Enhanced the data masking capability to be dynamic, allowing masking by word.

Previously, the microservice only allowed masking parts of the string at the beginning, end, or middle. With this update, it is now possible to mask each word individually according to business requirements.

Example of previous functionality: "Seb*** ****** *****"
Example of current functionality: "Seb**** Cue*** Nav***"

This was achieved by implementing logic that identifies and masks each word in the string according to the new specifications.
@JEZAPAT JEZAPAT force-pushed the feature/5309738-multi-mask branch from 1cc36c5 to fed5ab5 Compare May 28, 2024 19:44
@JEZAPAT JEZAPAT requested a review from gabheadz May 28, 2024 19:46
@JEZAPAT
Copy link
Contributor Author

JEZAPAT commented May 29, 2024

Please use conventional commits https://www.conventionalcommits.org/en/v1.0.0/ to describe changes made in the commits.

done

@JEZAPAT JEZAPAT closed this May 29, 2024
@JEZAPAT JEZAPAT reopened this May 29, 2024
Copy link
Contributor

@gabheadz gabheadz left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@gabheadz gabheadz merged commit 1c841b3 into bancolombia:master May 30, 2024
4 checks passed

🎉 This PR is included in version 1.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants