sbernhard
(Bernhard Suttner)
26 أبريل 2020، 8:17م
1
في الإصدارات السابقة، كانت تُعرض صفحة هبوط يظهر فيها زر “تسجيل الدخول”. عند النقر على هذا الزر، كانت تظهر صفحة تسجيل الدخول عبر SAML.
شخصيًا، أعجبتني هذه السلوكية كثيرًا. أما في الإصدار الحالي 2.4.2، لم يعد ذلك ممكنًا، حيث تُعرض صفحة تسجيل الدخول عبر SAML مباشرةً.
هل يمكن تكوين هذا السلوك؟
sam
(Sam Saffron)
27 أبريل 2020، 1:36ص
2
أنا لا أتبع، هل تقصد أننا عرضنا نافذة منبثقة تحتوي على زر واحد “تسجيل الدخول باستخدام SAML”؟
sbernhard
(Bernhard Suttner)
27 أبريل 2020، 6:46ص
3
لا. في إصدار سابق (أعتقد أنه 2.3.6) كان الأمر على هذا النحو:
بعد النقر على “Anmelden”، ظهرت نافذة تسجيل الدخول عبر SAML.
إذا قمت بتفعيل “تسجيل الدخول المحلي”، ستظهر صفحة الهبوط مع زر تسجيل الدخول مرة أخرى، لكنني لا أرغب في وجود تسجيل دخول محلي، وبالتالي أحتاج إلى تعطيله.
sbernhard
(Bernhard Suttner)
29 أبريل 2020، 9:21م
4
هل يمكن لأحد مساعدتي؟ ربما @eviltrout ؟
eviltrout
(Robin Ward)
29 أبريل 2020، 9:26م
5
بالتأكيد لم أقوم بتغيير ذلك عن قصد. إذا أمكنك مساعدتنا في تحديد وقت التغيير، فسيكون ذلك مفيدًا حيث يمكننا البحث في السياق.
من الممكن أن يكون التغيير قد حدث لأسباب أمنية.
Falco
(Falco)
29 أبريل 2020، 9:36م
6
كان هذا تحسينًا طالما طُلب لجعل omniauth يتصرف مثل SSO حقيقي، عندما يكون عدد طرق تسجيل الدخول عبر omniauth واحدًا فقط.
This is now implemented:
If there is only one external authenticator enabled and local logins are disabled and the site requires login, then users will be directed straight to the external authentication page. This exactly matches the implementation of our Discourse-native SSO.
I can’t think of a reason why anyone wouldn’t want this, so this is now the default behaviour.
cc @consideRatio @Joralf
sbernhard
(Bernhard Suttner)
29 أبريل 2020، 9:36م
7
كيف يمكنني المساعدة؟
تلميح واحد: إذا قمت بتفعيل “تسجيل الدخول المحلي”، فسيتم عرض “صفحة الهبوط مع زر تسجيل الدخول”.
david
(David Taylor)
29 أبريل 2020، 9:38م
8
إذا كنت تريد حقًا صفحة التأكيد، يمكنك ربط المستخدم بـ /login (بدلاً من الصفحة الرئيسية). هذا لن يؤدي إلى تسجيل الدخول تلقائيًا.
ولكن لاحظ أن هذا غير مدعوم فعليًا، بل هو مجرد عيب في التنفيذ وقد يتغير في أي وقت.
Falco
(Falco)
29 أبريل 2020، 9:39م
9
السلوك الجديد أفضل ومتسق مع تسجيل الدخول الموحد (SSO). إذا كانت هناك حاجة لرسالة قبل تسجيل الدخول، فيمكن تنفيذها في مزود الهوية.
sbernhard
(Bernhard Suttner)
29 أبريل 2020، 9:42م
10
آه، من الجيد معرفة ذلك.
أعرف سببين:
عرض صفحة هبوط جميلة تحتوي على بعض المعلومات الأساسية بدلاً من نافذة تسجيل الدخول “المملة” تلك
“الأمان” (أعلم أنها ليست عائقًا كبيرًا حقًا). من المأمول أن تفشل نصوص الاختراق الرخيصة على العنوان الرئيسي (بدون صفحة الهبوط) نظرًا لضرورة الضغط على زر تسجيل الدخول أولاً.
هل من الممكن تكوين هذا السلوك؟
sam
(Sam Saffron)
30 أبريل 2020، 2:29ص
11
ستحتاج إلى توظيف مطور هنا لبناء إضافة لتغيير ذلك. كما هو الحال، هذه هي الشكوى الأولى التي أسمعها حول النظام الجديد منذ أشهر.
sbernhard
(Bernhard Suttner)
30 أبريل 2020، 6:41ص
12
أنا مطور :-)(انظر إلى تغييراتي في discourse_saml)
هل تقبل طلب سحب (PR) يضيف خيارًا للإعدادات (الافتراضي: السلوك الجديد)؟
david
(David Taylor)
30 أبريل 2020، 8:48ص
14
لا ينبغي أن يؤدي إعادة التوجيه التلقائي إلى إدخال أي ثغرات أمنية. إذا كنت على علم بأي استغلال، يرجى إبلاغنا من خلال برنامج الإفصاح الخاص بنا: discourse/docs/SECURITY.md at main · discourse/discourse · GitHub
لا أعتقد أننا نريد إضافة المزيد من الضجيج إلى الإعدادات حتى يكون لدينا بعض المستخدمين الآخرين الذين يطلبون هذا التغيير.
يمكنك تجاوز هذا في إضافة عن طريق تعديل هذه الطريقة:
لذلك أعتقد أنك ستريد تبسيط الدالة إلى ما يلي:
def redirect_to_login
dont_cache_page
cookies[:destination_url] = destination_url
redirect_to path("/login")
end
sbernhard
(Bernhard Suttner)
30 أبريل 2020، 8:56ص
15
شكرًا لك. نقدر ذلك كثيرًا!
بالمناسبة: تقوم بعمل رائع مع Discourse.