دفعنا فلوس لتقرير مراجعة الكود وجاء مليان مصطلحات، كيف نقرأ تقرير مراجعة الكود؟

ركّز على البنود الأمنية المصنّفة "Critical" و"High" في التقرير. خصوصًا لو فيه نتائج داخلة في قائمة OWASP Top 10 (مثل التفويض المعطوب أو SQL injection) فهذي مش روائح كود، هذي أبواب تسريب بيانات قد تشكّل جريمة مباشرة. ما فيها تنازل.

الشركة اللي سوّت التدقيق مدينة لك بجلسة أسئلة وأجوبة 30 دقيقة. اسألهم مباشرة: "لو هذا النظام طلع بكرة للحيّاة ودخل 1.000 شخص في نفس اللحظة، أي 3 بنود بتفجّرنا أول؟" وأعطِ الوكالة هذي الـ3 بنود بس.

شركة التدقيق اللي دفعت لها 4.000 جنيه إسترليني غالبًا شغّلت أداة فحص آلية مفتوحة المصدر على الكود ومرّرت لك الـPDF اللي طلع بشعارهم. متأكد إنهم فعلًا فحصوا أخطاء المنطق يدويًا؟

في مشروعي الأول كدت أروح للمحكمة مع الوكالة بسبب تقرير مشابه. كنت أظن الأماكن المكتوب فيها "رائحة كود" إن النظام بيطير في أي لحظة. طلع إن الأداة بس نبّهت لأن أسماء الدوال ما تطابق قواعد الإنجليزية تمامًا. قبل ما تدبّس، خلّي خبير يترجمه لك.

فلتر البنود بهذا الترتيب: 1) الثغرات المتعلقة بأمان قاعدة البيانات والعملاء فورًا، 2) الاستعلامات اللي تسبب أقفالًا في العمليات المتزامنة في السبرنت القادم، 3) النقص في الشكليات والتوثيق يُحل تمامًا في أوقات الفراغ.

لو رميت التقرير كامل على الوكالة بيتحولون للدفاع وما بيسوون شي لكن بصراحة خذ سكرين شوت للأخطاء الحقيقية بس وقل لهم هذي مخالفة لمعايير التسليم وأرسلها.

اللي يسمونه رائحة كود، هو مصطلح متعلق بحرارة السيرفر الزايدة أو احتراق العتاد، ولا تعبير مجازي يستخدمه المبرمجون بينهم؟

ملخص سريع للجدد: الخطأ المرتكب من جانب تقرير مراجعة الكود قابل للإصلاح عادةً، لكنه مكلف.

دعني أتركها كملاحظة، قد تكون مفيدة.

عانيت من نفس المشكلة قبل عامين. الحلول التي تنجح على نطاق صغير تنهار عند التوسع، تعلمت هذا متأخراً.

هذه وجهة نظري، ولا أدعي أنها الحقيقة المطلقة.

سأجرب ذلك.

موضوع في الوقت المناسب تمامًا.

تشتت النقاش، دعني ألخصه. الناس يدافعون عن العادات وليس العمليات. المقاومة تأتي من هناك.

نحن أيضاً واجهنا نفس المشكلة في فترة معينة. وضع عملية لطلبات التغيير لا يبطئ العمل، بل يسرّعه.

الخطأ المرتكب من جانب تقرير مراجعة الكود قابل للإصلاح عادةً، لكنه مكلف.

زالت حيرتي، شكراً لكم.

تم الحفظ.

أكثر نقطة يتم إغفالها في موضوع تقرير مراجعة الكود هي: يجب فتح بيانات الوصول باسم المؤسسة، وليس بحسابات شخصية.

أنت محق.

هذا العنوان مؤرشف.