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

Not using Crypt #5

Closed
mellester opened this issue May 18, 2021 · 1 comment
Closed

Not using Crypt #5

mellester opened this issue May 18, 2021 · 1 comment

Comments

@mellester
Copy link

mellester commented May 18, 2021

Looking at the source code you directly implement your own Encrypted class.
Whilst the readme says your are using the Crypt class
This class does not use a initialization vector so therefore is less secure than they default Crypt class from Laravel.
In short this means duplicate values will have the same value in the database

@elgibor-solution
Copy link
Collaborator

elgibor-solution commented Aug 6, 2021

Hi @mellester

Yeah, If use crypt then i can't run search query, i will update the README. Feel free to do PR if you have a better idea..

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

No branches or pull requests

2 participants