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
Fix security issue for V4 #2424
base: v4
Are you sure you want to change the base?
Conversation
Hi @viveleltsi, thanks for your code. Your v4 source, did you get from v4 branch or from v4.1.4 tag? |
Hello ! Thank you for your response. My branch was created on the v4.1.4 tag. (commit 918b650). |
d3182c2
to
3f7eac9
Compare
I rebase my branch onto the V4 branch (commit 46a8387) and I rebuild it and run the unit testing. It's all good. |
Nice! Thanks.... Could you create a pull request for v4 branch? Than I will create a new tag for this new release |
Yes of course. I updated this PR to target the v4 branch. |
@mbdavid I updated the PR to the v4 branch :) |
Fix the severity issue for the old v4 liteDB version.
The goal is to have a patch for application who cannot update to V5.
Because this is my first PR for this project I think my changes should be properly checked and validated.
fix #2418
PS: This branch should not be merged on master but probably stay on a fix branch.