On a payé pour une revue de code et c'est bourré de jargon, comment lire un rapport de revue de code ?

Focalisez-vous sur les alertes de sécurité taguées en "Critical" et "High". Surtout si certaines figurent dans le Top 10 de l'OWASP (genre contrôle d'accès défaillant ou injection SQL) : ce n'est pas du simple code mal écrit, ce sont de vraies portes ouvertes à des fuites de données qui peuvent engager votre responsabilité légale. Là-dessus, aucun compromis.

Le cabinet d'audit vous doit bien un call de débrief de 30 minutes. Posez-leur directement la question : "Si on lance l'appli demain et qu'il y a 1 000 utilisateurs connectés en même temps, quels sont les 3 premiers trucs qui vont faire tout exploser ?" Donnez uniquement ces 3 points-là à votre agence.

Pour 4 000 livres, il y a de fortes chances que votre cabinet d'audit ait juste lancé un outil d'analyse automatique open source sur le code et vous ait refourgué le PDF avec leur logo dessus. Vous êtes sûr qu'ils ont vraiment audité la logique métier à la main ?

Dans ma première boîte, j'ai failli finir au tribunal avec une agence à cause d'un rapport de ce genre. Je voyais "code smell" partout, je croyais que le système allait s'effondrer. En fait, c'était juste l'outil qui râlait parce que le nom des fonctions n'était pas parfait en anglais. Faites-vous expliquer le rapport par un expert avant de paniquer.

Traitez les retours dans cet ordre : 1) Les failles touchant à la base de données et à la sécurité des clients, immédiatement. 2) Les requêtes qui provoquent des blocages lors des accès concurrents, au prochain sprint. 3) Les soucis de forme et de documentation, uniquement quand vous aurez du temps creux.

si vous balancez tout le rapport brut a l'agence ils vont se braquer et rien faire. prenez juste des captures des vrais bugs bloquants et envoyez-les en disant que ca respecte pas le cahier des charges.

ce qu'ils appellent "code smell", c'est lié à une surchauffe des serveurs ou à du matériel qui crame ou c'est juste une métaphore entre développeurs ?

Résumé rapide pour les nouveaux : L'erreur commise du côté de rapport de revue de code est généralement réversible mais coûteuse.

Je le note au cas où.

J'ai vécu exactement la même chose il y a deux ans. Les solutions qui fonctionnent à petite échelle s'effondrent en grandissant, j'ai appris cela trop tard.

C'est mon avis, je ne l'écris pas comme une vérité absolue.

Je vais essayer.

Sujet très actuel.

La discussion part dans tous les sens je recentre. Les gens défendent l'habitude, pas le processus. La résistance vient de là.

Nous avons aussi bloqué au même endroit à une époque. Mettre en place un processus de demande de changement n'ralentit pas le travail, il l'accélère.

L'erreur commise du côté de rapport de revue de code est généralement réversible, mais coûteuse.

Mes doutes sont levés, merci.

enregistré.

Le point le plus souvent négligé concernant rapport de revue de code est : Les accès doivent être créés au nom de l'entreprise, pas avec des comptes personnels.

Vous avez raison.

ce sujet est archivé.