#36 feat: login diretto SISTER per intestatari della convenzione (uso personale)

[Anonymous]

Originally created by: zornade
Originally owned by: zornade

Risolve [#35].

Cosa cambia

Aggiunge il provider SPID_PROVIDER=sister al dispatcher introdotto in [#31] (provider Poste): login diretto al portale SISTER tramite il tab dedicato sulla pagina ADE, con credenziali nominali della convenzione.

Riepilogo tecnico

• utils._login_sister_direct(page, logger, username, password) — clicca il tab "Sister", inserisce SISTER_USERNAME/SISTER_PASSWORD, clicca "Accedi", attende networkidle.
• utils.login() — branching dopo goto_login: se SPID_PROVIDER=sister chiama direttamente _login_sister_direct() e ritorna, bypassando "Entra con SPID" e l'intera navigazione ADE portal (cerca SISTER → Vai al servizio → Conferma → Consultazioni → Visure catastali → Conferma Lettura), non necessaria perché il login SISTER nominale atterra già nell'area servizio.
• Gestione sessioni orfane: fino a 10 tentativi di CloseSessionsSis + ri-login automatici prima di alzare l'eccezione (il sistema SISTER blocca quando rileva una sessione precedente non chiusa correttamente).
• Test: 3 nuovi test per il dispatcher SISTER (credenziali mancanti, case-insensitive, messaggio errore valori validi).

Scope d'uso esplicito (anche in README e docstring)

Il login diretto SISTER è destinato esclusivamente all'intestatario della convenzione SISTER che desidera automatizzare le proprie consultazioni con le proprie credenziali nominali. Non è destinato a chi vuole rivendere o esporre l'accesso a terzi: la convenzione SISTER richiede che l'utenza sia personale, non cedibile, e che le consultazioni siano riconducibili all'intestatario. La responsabilità dell'uso delle credenziali resta dell'intestatario, esattamente come per il flusso SPID.

Documentazione aggiunta in README.md, sezione Provider di autenticazione supportati, e nella docstring di _login_sister_direct.

Dipendenza

Questa PR si basa su feat/spid-provider-poste (#31): se [#31] viene mergiato prima, GitHub rebaserà automaticamente questa PR su main. Se viene mergiato dopo, il rebase è banale (estende lo stesso dispatcher).

Provenienza upstream

Cherry-pick di:

• pietromalerba/visura-api@6bcd072 — estrazione _login_sister_direct + dispatcher
• pietromalerba/visura-api@b2d606d — retry CloseSessionsSis fino a 10 tentativi

Adattamenti rispetto all'upstream:

• merge con il dispatcher SPID Poste introdotto in [#31] (l'upstream conosce solo sielte+sister; qui supportiamo sielte+poste+sister)
• ampliamento docstring con scope d'uso ammesso
• aggiunta sezione README dedicata
• test estesi per coprire anche il provider sister

Compatibilità

Retrocompatibile: default invariato SPID_PROVIDER=sielte. Nessun cambiamento per chi usa SPID Sielte o Poste.

Test

PYTHONPATH=. ADE_USERNAME=test ADE_PASSWORD=test pytest tests/ -q
49 passed

Lint:

black --check .  → All done
ruff check .     → All checks passed

Check di review

• [x] DCO sign-off su tutti i commit
• [x] (cherry picked from commit ...) su entrambi i pick (-x)
• [x] Co-author = Pietro Malerba (preservato dal cherry-pick)
• [x] CI verde locale (black + ruff + 49 test)
• [x] Issue collegata (#35) con framing esplicito
• [x] README aggiornato con scope d'uso

[Anonymous]

• assigned_to: zornade

[Anonymous]

Ticket changed by: zornade

• status: open --> closed