You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
If you call recognize without limit parameter the url is not valid and a 400 is returned from the REST API.
To Reproduce
try{constresult=awaitrecognitionService.recognize(path_toTestImage,{face_plugins: 'landmarks,gender,age',det_prob_threshold: 0.95,status: true});}catch(e){console.error('Error while recognize'+e.message);console.error(e.config.url);// <-- comes here. not good}
Prints http://localhost:8000/api/v1/recognition/recognize?det_prob_threshold=0.95?face_plugins=landmarks,gender,age?status=true
Expected behavior
If I do not pass limit it should use a default or build a valid url.
Should be http://localhost:8000/api/v1/recognition/recognize?det_prob_threshold=0.95&face_plugins=landmarks,gender,age&status=true
Describe the bug
If you call
recognize
without limit parameter the url is not valid and a 400 is returned from the REST API.To Reproduce
Prints
http://localhost:8000/api/v1/recognition/recognize?det_prob_threshold=0.95?face_plugins=landmarks,gender,age?status=true
Expected behavior
If I do not pass limit it should use a default or build a valid url.
Should be
http://localhost:8000/api/v1/recognition/recognize?det_prob_threshold=0.95&face_plugins=landmarks,gender,age&status=true
Maybe other webservers/backends do not care but I use the docker-compose version from https://github.com/exadel-inc/CompreFace/releases/tag/v1.0.0
The text was updated successfully, but these errors were encountered: