يجب أن يؤدي النقر على الظرف للكشف عن بريد إلكتروني وارد إلى العمل للموظفين وللمستخدمين الذين يعرضون منشوراتهم الخاصة. ومع ذلك، كمستخدم غير موظف يحاول عرض منشوره الخاص، يؤدي هذا إلى خطأ في جافاسكريبت. يمكن للمستخدم عرض ملف JSON الأساسي عندما يتم طلب ذلك يدويًا.
يمكنني تأكيد أن هذا يحدث أيضًا في موقع الاختبار الخاص بي للمستخدمين غير الموظفين بنفس الخطأ - Uncaught Error: Failed to create an instance of 'controller:raw-email'. Most likely an improperly defined class or an invalid module export.
يبدو شرعيًا! شكرًا على طرحه، نظرًا لأن القطع الورقي صغير جدًا، فسأضع عليه علامة pr-welcome. أنا أؤيد الإصلاح البسيط للغاية المتمثل في عدم استدعاء أي شيء هنا، وأنا متردد بشأن الكشف عن معلومات البريد الإلكتروني الكاملة للذات لغير المشرفين.
لعدم استدعاء أي شيء؛ لقد استخدمت ذلك لتصحيح مشاكل البريد الإلكتروني مرات لا تحصى كمسؤول، ولم ألقِ نظرة على رسائلي الخاصة كمشرف أو مستخدم عادي، في أي مكان.
يمكنني قبول عدم وجود اختبار للجانب العميل إلى حد ما، ولكن يجب أن يكون اختبار جانب واجهة برمجة التطبيقات (API) سهلاً. تقوم بإنشاء كائن منشور، وكائن مستخدم، وتتحقق من أن المستخدم لا يمكنه عرض البريد الإلكتروني الخام عند حذف المنشور في اختبارات الحارس.
سأحاول معرفة سبب إغلاق طلب السحب (PR).
هذا تغيير في السلوك، أنا فضولي لماذا نتجه نحو إزالة الميزة بدلاً من إصلاح الخلل.
هذا غريب بالتأكيد لأنني لا أعتقد أنه كان مقصودًا إغلاقه ولم أكن أنا من أغلقه. على الأقل ليس عن قصد وليس على حد علمي. على أي حال، سأتابع طلب السحب قريبًا.
هناك شيء غريب في حالة طلب السحب. خطأ في GitHub ربما؟