-
Notifications
You must be signed in to change notification settings - Fork 229
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
Remote image classifier #2439
base: master
Are you sure you want to change the base?
Remote image classifier #2439
Conversation
Thank you very much @hauck-jvsh! Just a couple a quick suggestions:
|
I didn’t see that the compression was on, I Will change that. I will adjust the other suggestions to. |
I think we could also encrypt thumbs before sending them since this will handle CSAM cases... Maybe we could create a new config parameter to set the server assimetric public key, so the client could send a simetric session key and we wouldn't have to deal with certificates? |
Make zip an inner class
I think that it is finished now, I use the same method of the DIE task to combine the scores of multiple frames of video/gifs files in to one. I just need to enable the https to encrypt all thumbs |
@lfcnassif I just added the option to skip the validation of the ssl's certify. |
No description provided.