deis / deis/workflow-cli

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

Offen
#24 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement security
Vorherrschende Sprache
Go
Sterne
31
Forks
42
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne am clientseitigen Attach-Einstiegspunkt, der von `certs:attach ` verwendet wird, und finde den bestehenden Unit-Test zum Anhängen von Zertifikaten. Führe den gezeigten Aufruf mit `non-existent.domain.com` erneut aus, sorge dann dafür, dass der Befehl für diesen Fall einen Fehler zurückgibt, und verifiziere, dass der Test erfolgreich ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
go
Bereich
cli
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.