Project

General

Profile

Segnalazione #133

fuss-server cp non chiede i parametri di configurazione

Added by Simone Piccardi about 7 years ago. Updated about 7 years ago.

Status:
Chiuso
Priority:
Normale
Start date:
01/10/2017
Due date:
% Done:

0%

Estimated time:

Description

Dopo aver lanciato fuss-server create, un successivo lancio di fuss-server cp non chiede i parametri di configurazione e fallisce sul task:

 cp-common : calculate uamlisten value from shell

History

#1

Updated by Elena Grandi about 7 years ago

  • Status changed from Nuovo to Commenti
  • Assignee changed from Elena Grandi to Simone Piccardi

Giusto, avevo lasciato i valori facoltativi per permettere di non configurarli quando non c'è il captive portal, ma qui servono.

è a posto?

(è possibile che non funzioni se gli si passa un file vecchio che non ha i due campi: è un problema, o comunque il file base sarà sempre quello nuovo e quindi ce li ha?)

#2

Updated by Simone Piccardi about 7 years ago

  • Assignee changed from Simone Piccardi to Elena Grandi

Elena Grandi ha scritto:

Giusto, avevo lasciato i valori facoltativi per permettere di non configurarli quando non c'è il captive portal, ma qui servono.

è a posto?

I parametri devono essere facoltativi quando si lancia fuss-server create, ma se si lancia fuss-server cp significa che si vuole creare un captive portal, ed in quel caso se non ci sono devono essere chiesti (e deve essere verificato che si possano usare, tipo controllare che ci sia una interfaccia disponibile).

(è possibile che non funzioni se gli si passa un file vecchio che non ha i due campi: è un problema, o comunque il file base sarà sempre quello nuovo e quindi ce li ha?)

Sarà un problema se qualcuno farà i test su una macchina su cui ha installato una versione vecchia del fuss-server che non li prevede. Non è necessario curarlo aggiungendo orpelli al programma, ma occorrerà mettere un avviso molto grosso quando si farà il rilascio della versione.

Il caso d'uso che prima si installa il fuss-server e poi il captive portal deve continuare a funzionare, ho provato comunque, piallando /etc/fuss-server/fuss-server.yaml a eseguire fuss-server cp e non funziona lo stesso (non ho la terza interfaccia sulla macchina su cui l'ho provato, ma non mi ha dato nessun errore ne mi ha chiesto i parametri di configurazione).

#3

Updated by Elena Grandi about 7 years ago

  • Assignee changed from Elena Grandi to Simone Piccardi

Simone Piccardi ha scritto:

Elena Grandi ha scritto:

è a posto?

I parametri devono essere facoltativi quando si lancia fuss-server create, ma se si lancia fuss-server cp significa che si vuole creare un captive portal, ed in quel caso se non ci sono devono essere chiesti (e deve essere verificato che si possano usare, tipo controllare che ci sia una interfaccia disponibile).

infatti, è la modifica che ho fatto e di cui chiedevo se fosse a posto

Sarà un problema se qualcuno farà i test su una macchina su cui ha installato una versione vecchia del fuss-server che non li prevede. Non è necessario curarlo aggiungendo orpelli al programma, ma occorrerà mettere un avviso molto grosso quando si farà il rilascio della versione.

Per il commento di Christopher su #127 ho comunque aggiunto un controllo che il file di configurazione abbia tutti i campi necessari (anche vuoti), per essere più solidi in caso di file che viene danneggiato (si limita a segnalare il problema e suggerisce di cancellare il file per ripartire da zero, comunque)

Il caso d'uso che prima si installa il fuss-server e poi il captive portal deve continuare a funzionare, ho provato comunque, piallando /etc/fuss-server/fuss-server.yaml a eseguire fuss-server cp e non funziona lo stesso (non ho la terza interfaccia sulla macchina su cui l'ho provato, ma non mi ha dato nessun errore ne mi ha chiesto i parametri di configurazione).

Uhm, questo non mi quadra: al momento lanciando cp dovrebbe

  • controllare individualmente che i due valori ci siano (non vuoti), che l'interfaccia sia presente (circa riga 281) e che la network sia effettivamente una network (riga 299)
  • dentro a _crosscheck_hotspot (riga 174) fare controlli incrociati che i due valori siano o entrambi pieni o entrambi vuoti (dato che deve funzionare anche nel caso senza cp) e poi:
    • che l'interfaccia non sia la stessa di una di quelle configurate per LAN e WAN
    • che l'interfaccia non sia un'interfaccia tun
    • che la network non sia già in uso sulla macchina

(alcune di queste parti sono coperte dai testi in fuss-client selftest, ma ovviamente non quelli che dipendono dalla configurazione della macchina)

C'è un bug per cui qualcosa non viene controllato? Cosa?

(Dato che prima mi sono accorta di aver scritto sul ticket prima di completare il push, io al momento sono sul 4630cef1)

#4

Updated by Simone Piccardi about 7 years ago

  • Status changed from Commenti to Chiuso

Elena Grandi ha scritto:

Simone Piccardi ha scritto:

Elena Grandi ha scritto:

è a posto?

I parametri devono essere facoltativi quando si lancia fuss-server create, ma se si lancia fuss-server cp significa che si vuole creare un captive portal, ed in quel caso se non ci sono devono essere chiesti (e deve essere verificato che si possano usare, tipo controllare che ci sia una interfaccia disponibile).

infatti, è la modifica che ho fatto e di cui chiedevo se fosse a posto

Mi ero perso che la modifica era stata fatta.

Per il commento di Christopher su #127 ho comunque aggiunto un controllo che il file di configurazione abbia tutti i campi necessari (anche vuoti), per essere più solidi in caso di file che viene danneggiato (si limita a segnalare il problema e suggerisce di cancellare il file per ripartire da zero, comunque)

Va bene così allora.

Il caso d'uso che prima si installa il fuss-server e poi il captive portal deve continuare a funzionare, ho provato comunque, piallando /etc/fuss-server/fuss-server.yaml a eseguire fuss-server cp e non funziona lo stesso (non ho la terza interfaccia sulla macchina su cui l'ho provato, ma non mi ha dato nessun errore ne mi ha chiesto i parametri di configurazione).

Uhm, questo non mi quadra: al momento lanciando cp dovrebbe

Avevo ritestato la versione prima dei cambiamenti. Così funziona e chiede il necessario, poi si ferma per la mancanza di coova-chilli sul repository, ma questo è un altro problema.

Also available in: Atom PDF