search

SIWECOS / siwecos-business-layer

SIWECOS Main API and Business Layer Application
https://siwecos.de
0 stars 1 forks source link

Bessere Verifizierungs Rückmeldung #160

Closed Skeeve closed 4 years ago

Skeeve commented 4 years ago

Wir brauchen bessere Meldungen an die Nutzer, wenn eine Verifizierung einer Domain fehlschlägt.

Siehe #159

Wenn hier die Meldung "Ihre Domain XXX gewährt uns keinen Zugriff. Die Meldung war: 403 Forbidden" erschienen wäre, hätten wir schneller die Ursache finden können.

Lednerb commented 4 years ago

Bereits realisiert:

https://github.com/SIWECOS/siwecos-business-layer/blob/d5381d625e2ece73dd674867cc16d74f9416db22/app/DomainVerifier.php#L26-L38

Siehe Kibana Logging -> gespeicherte Suche: "Domain-Validierungsfehler"

Skeeve commented 4 years ago

Leider nicht realisiert.

Der Nutzer hat seinen 403 nicht gesehen, weil Du keine Exception wirfst sondern 'false' zurückgibst.

https://github.com/SIWECOS/siwecos-business-layer/blob/d5381d625e2ece73dd674867cc16d74f9416db22/app/DomainVerifier.php#L45-L53

Lednerb commented 4 years ago

Im oben genannten Fall hätte der Nutzer allerdings auch mit der 403er Nachricht nichts anfangen können.

"Bereits realisiert" war auf das wir bezogen:

Wenn hier die Meldung "Ihre Domain XXX gewährt uns keinen Zugriff. Die Meldung war: 403 Forbidden" erschienen wäre, hätten wir schneller die Ursache finden können.