fix: remplace cv url by cv binary

[Abdellahitech]

No description provided.

[the-forest-tree]

Le pattern except Exception est à enlever partout. Est ce que je t'ai pas déjà expliquer que c'est une pratique dangereuse ?

[the-forest-tree]

Non ca c'est trop excep Exception ca va tout catcher. Ca veut dire si par exemple dans ton code a un moment tu fais une erreur du genre

try: 
    binary_data = base64.b64decodeX(cv_base64) <==== le X à la fin de ba64decode 

Tu vas rien voir. Soubhana Lah. Tu trouves ca normal ?

Tu vas raiser en dessous Base64DecodeError comme si c'est un problème dans la data alors que en fait c'est une erreur. La méthode b64decodeX n'existe pas il faut enlever le X

Le message c'est que faut quasiment jamais faire except Exception c'est trop dangereux. Donc la soit tu vas chercher les erreur suceptible d'être raisé du genre si tu fais json.loads(SOME_USER_DATA_THAT_MIGHT_BE_MALFORMATED) tu peux faire except JSONDecodeError ici c'est justifié.

Mais except Exception non c'est trop

[Abdellahitech]

ok j'ai enlevé le catch des erreurs. en cherchant j'ai trouvé qu'il y a une erreur possible que le len soit pas divisible par 4 donc à ajouter des 4. c'est seulement l'erreur que j'ai catché

[the-forest-tree]

Même commentaire par rapport à except Exception
Tu remplaces ca par l'erreur qui peut être raisé en circonstance normale

[sentry-io]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ ValueError: No base64 string provided for CV. workflow-executor View Issue

_{Did you find this useful? React with a 👍 or 👎}