تحميل الوسائط الآمن يكسر شعارات الفئات

أعتقد أن المشكلة الرئيسية هنا هي أن دالة secure_upload_analyse_and_update قاسية بعض الشيء عند تحديد ما يجب أن يكون آمناً وما لا يجب، لأن فئة UploadSecurity (discourse/lib/upload_security.rb at main · discourse/discourse · GitHub) تتحقق من مكان وجود هذه الأنواع العامة من التحميلات (مثل صور الرموز الشخصية، وشعارات التصنيفات، وما إلى ذلك) سواء يجب أن تكون آمنة أم لا وقت التحميل، وهذا type غير موجود عند إعادة التحقق في مهمة rake.

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

على أي حال، أقوم حالياً ببناء طلب دمج (PR) لحل هذه المشكلة بجعل شعارات التصنيفات والخلفيات من الأنواع العامة عند النظر في ما إذا كان يجب أن تكون آمنة أم لا. بمجرد دمج هذا الطلب وتحديثك، ستحتاج فقط إلى إعادة تحميل صور التصنيفات، وستكون الأمور على ما يرام.

4 إعجابات