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

Aberta

#4 aberta em 8/12/2023 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade: 2/5
Tempo estimado: 1-3 horas
Facilidade para iniciantes: 35/100

Tipo de issue: Bug
Clareza: Razoavelmente clara
Status de atividade: Estagnada
Stack de tecnologia: php
Domínio: backend

Direção de pesquisa

Comece em src/Controller/ConsentAdmin.php na linha vinculada e inspecione como cada entrada de consent.disable é tratada quando contém a configuração de regex mostrada. Reproduza a falha com essa configuração e, em seguida, verifique se o módulo não falha mais e se a filtragem de serviços pretendida continua funcionando.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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]);
                }
            }
        }

Linguagem predominante: PHP
Estrelas: 4
Forks: 1
Métricas de merge de PRs: Nenhum PR com merge em 30d

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Primeiros passos

Leia a issue inteira e depois o guia de contribuição do projeto.
Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
Faça um fork do repositório e trabalhe em uma branch.
Abra um pull request que referencie o número da issue.

Issues semelhantes

[Sync EN] stream_copy_to_stream: document offset and socket timeout behavior (#5860) Aberta

sync-en

Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100

php/doc-fr#3579 · Aberta em 25/9/2026
[Sync EN] stream_copy_to_stream: document offset and socket timeout behavior (#5860) Aberta

sync-en

Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100

php/doc-es#1338 · Aberta em 25/9/2026
[Sync EN] Document Dom\Element::getElementsByClassName() for PHP 8.5 Aberta

Перевод устарел

Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100

php/doc-ru#1893 · Aberta em 25/9/2026
Paragraphs module default install change Aberta

bug good first issue

Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100

az-digital/az_quickstart#6047 · Aberta em 24/9/2026
Error when saving email Aberta

bug needs-triage

Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100

mautic/mautic#17533 · 1 comentário · Aberta em 24/9/2026

Mais issues de PHP