Open caglaroflazoglu opened 3 years ago
Merhabalar.
SimpleSAMLphp-1.19.1 kaynak kodlarında identifyingAttribute ve Consent modülü ile ilgili bir kod parçacığına rastlamadım. Ayrıca 1.19.1 ile kendimiz bir kurulum yaptık ve consent modülü düzgün bir şekilde çalışıyor. Siz acaba github'dan güncel kaynak kodları indirerek mi kurulum yaptınız?
Büyük olasılıkla bahsettiğiniz değişiklikler 1.19.2'de ya da 2.0.0'da aktif olacaktır. O sürümler için gerekli güncellemeleri ileride yapabiliriz.
uğur
Merhabalar.
SimpleSAMLphp-1.19.1 kaynak kodlarında identifyingAttribute ve Consent modülü ile ilgili bir kod parçacığına rastlamadım. Ayrıca 1.19.1 ile kendimiz bir kurulum yaptık ve consent modülü düzgün bir şekilde çalışıyor. Siz acaba github'dan güncel kaynak kodları indirerek mi kurulum yaptınız?
Büyük olasılıkla bahsettiğiniz değişiklikler 1.19.2'de ya da 2.0.0'da aktif olacaktır. O sürümler için gerekli güncellemeleri ileride yapabiliriz.
uğur
github dan master clone u
identifyingAttribute için Consent kod satırını yukarıda paylaşmıştım.
Merhabalar. SimpleSAMLphp-1.19.1 kaynak kodlarında identifyingAttribute ve Consent modülü ile ilgili bir kod parçacığına rastlamadım. Ayrıca 1.19.1 ile kendimiz bir kurulum yaptık ve consent modülü düzgün bir şekilde çalışıyor. Siz acaba github'dan güncel kaynak kodları indirerek mi kurulum yaptınız? Büyük olasılıkla bahsettiğiniz değişiklikler 1.19.2'de ya da 2.0.0'da aktif olacaktır. O sürümler için gerekli güncellemeleri ileride yapabiliriz. uğur
github dan master clone u
identifyingAttribute için Consent kod satırını yukarıda paylaşmıştım.
Merhabalar.
Github'dan son sürümü çekmek yerine kararlı bir sürümü kullanmanızı tavsiye ederim. Kararlı yeni sürümde bahsedilen değişiklikler yapılacak olabilir (çok olası). Yeni kararlı sürümde, belgede de güncelleme yapılır.
uğur
Modül güncel olmadığı için yeni versiyon simplesamlphp ile sorun yaşatıyor. saml20-idp-hosted.php de yer alan tanımda
97 => [ 'class' => 'consent:Consent', 'store' => 'consent:Cookie', 'focus' => 'yes', 'checked' => false ], yeterli gelmiyor. Çünkü modülün
https://github.com/simplesamlphp/simplesamlphp-module-consent/blob/7026cc1cd7006e995860a346f843d439bc45b1e7/lib/Auth/Process/Consent.php#L177 Assert::keyExists($config, 'identifyingAttribute', "Consent: Missing mandatory 'identifyingAttribute' config setting.");
ifadesi geçiyor. Yukarıdaki yapılandırmada "identifyingAttribute" bileşeni yer almadığı için bu satırda hata alıyorsunuz. Bu nedenle ilk etapta config dosyasını şu şekilde yapılandırmak gerekti
97 => [ 'class' => 'consent:Consent', 'store' => 'consent:Cookie', 'focus' => 'yes', 'checked' => false, 'identifyingAttribute' =>'mail' // test amaçlı ],
yapılan bu ayar değişikliği ile sorun çözülüyor fakat, bu sefer de modül fonksiyonları eski kaldığı için hata almaya başlıyorsunuz.
https://github.com/simplesamlphp/simplesamlphp-module-consent/blob/7026cc1cd7006e995860a346f843d439bc45b1e7/www/getconsent.php#L134 $translator->t() fonksiyonu tanımsız uyarısı vermeye başlıyor.
Konuyla ilgili bir issue var. Tamamlanmış bir çözüm değil, php versiyonu ile ilişkili de olabilir. Php 8 de bazı tanımlar değişmişti. https://github.com/simplesamlphp/simplesamlphp-module-consent/issues/10
97 nolu anahtarı iptal ederek yolumuza devam ettik.