deis / deis/workflow-cli

determine if domain is valid in 'certs:attach <cert> <domain>'

Aperta
#24 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
enhancement security
Lingua principale
Go
Stelle
31
Fork
42
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Currently, although `certs:attach ` will properly error out if `` is bogus/non-existent, it doesn't if `` is similarly non-existent. Consider the following unit test, which is currently failing (err is nil):

```
if err := Attach(&client, "test-example-com", "non-existent.domain.com"); err == nil {
t.Fatal("An Error should have resulted from the attempt to attach a valid cert to a non-existent domain")
}
```

This issue tracks work on the client side, and deis/workflow#475 tracks server-side work

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia dal punto di ingresso Attach lato client utilizzato da `certs:attach ` e trova il test unitario esistente per l'associazione dei certificati. Riproduci la chiamata mostrata con `non-existent.domain.com`, fai in modo che il comando restituisca un errore per quel caso e verifica che il test abbia esito positivo.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
go
Ambito
cli
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.