It breaks when regex type in `consent.disable` exists

Offen

#4 erstellt am 08.12.2023 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: php
Bereich: backend

Rechercherichtung

Beginne in src/Controller/ConsentAdmin.php bei der verlinkten Zeile und untersuche, wie jeder Eintrag von consent.disable behandelt wird, wenn er die gezeigte regex-Konfiguration enthält. Reproduziere den Fehler mit dieser Konfiguration und überprüfe anschließend, dass das Modul nicht mehr fehlschlägt und die beabsichtigte Service-Filterung weiterhin funktioniert.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

When you have ['type' => 'regex', 'pattern' => '/.*\.mycompany\.com.*/i'], in consent.disable variable in config breaks the module.

The relevant line: https://github.com/simplesamlphp/simplesamlphp-module-consentadmin/blob/799cc2096c379a29fc90d1cd711e29718955a224/src/Controller/ConsentAdmin.php#L206

There is no type check for $disable variable.

        // Remove services, whitch have consent disabled
        if (isset($idp_metadata['consent.disable'])) {
            foreach ($idp_metadata['consent.disable'] as $disable) {
                if (array_key_exists($disable, $all_sp_metadata)) {
                    unset($all_sp_metadata[$disable]);
                }
            }
        }

Vorherrschende Sprache: PHP
Sterne: 4
Forks: 1
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.