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

DataBase Support Added face recognition database isssue #1363

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

viv2712
Copy link

@viv2712 viv2712 commented Aug 30, 2021

I have Created A database support in sqlite3. For Values directly from the database and compare with the face recognition software. And do identify using the ID name.
Issue: resolved Link.
URL:#1332

@viv2712 viv2712 changed the title DataBase Support DataBase Support Aug 30, 2021
@viv2712 viv2712 changed the title DataBase Support DataBase Support Fixeded this issue:URL:https://github.com/ageitgey/face_recognition/issues/1332 Aug 30, 2021
@viv2712 viv2712 changed the title DataBase Support Fixeded this issue:URL:https://github.com/ageitgey/face_recognition/issues/1332 DataBase Support Fixeded this issue: Aug 30, 2021
@viv2712 viv2712 changed the title DataBase Support Fixeded this issue: DataBase Support Added face recognition database isssue Aug 30, 2021
Copy link

@LYF123123 LYF123123 left a comment

Choose a reason for hiding this comment

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

Dear developer, maybe there is Database Injection Risk at Line 18,25,27 in dataset.py and Line 19 in detector.py
Maybe you can write like this cursor.execute('insert into user (name,password) value (?,?)',(name,password))

@viv2712
Copy link
Author

viv2712 commented Aug 30, 2021

ohk let me make changes and then I will let you know!!!

@viv2712
Copy link
Author

viv2712 commented Aug 30, 2021

here I am trying to do in both the files in those lines you have been suggested is geting the database table which I have created in my sqlite3 database and then if their is similar data present into then just don't execute the command and if the data is not present just update it and your code we are just inserting the data so we need to keep it in the similar way....If it shows error again and again then I look into it.....

@viv2712
Copy link
Author

viv2712 commented Aug 30, 2021

let me know which database your using or the snippet of yours..

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

Successfully merging this pull request may close these issues.

None yet

2 participants