Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scelta del livello di sicurezza #53

Open
matteonespoli opened this issue Sep 20, 2023 · 2 comments
Open

Scelta del livello di sicurezza #53

matteonespoli opened this issue Sep 20, 2023 · 2 comments
Assignees

Comments

@matteonespoli
Copy link

Buongiorno Marco,
proporrei l'introduzione di un parametro da settare in configurazione per scegliere il livello di sicurezza da utilizzare nell'accesso (1/2/3).
Per ora ho provveduto con la modifica del sorgente ma sicuramente è qualcosa che potrebbe essere utile impostare a tutti.

Grazie mille e buon lavoro!

@milesimarco
Copy link
Member

Ciao @matteonespoli,
potrebbe essere aggiunto come parametro nelle impostazioni o come hook.

A livello di codice, nel tuo test mi confermi che hai modificato la seguente linea aggiungendo un parametro ulteriore (1, 2 o 3)?

wp-spid-italia.php:287

$sp->login( 'idp_'.$_GET['spid_idp'], $assertId, $attrId );

A presto,
M.

@milesimarco milesimarco self-assigned this Oct 11, 2023
@matteonespoli
Copy link
Author

matteonespoli commented Nov 19, 2023

Ciao @milesimarco,
confermo. Così da variare l'attributo "level" nella funzione "login" della classe Saml. Originariamente statico a 1.

public function login(
        string $idpName,
        int $assertId,
        int $attrId,
        $level,
        string $redirectTo = null,
        $shouldRedirect = true
    ) 

Grazie mille!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants