Improve dns_get_record / dns_check_record error handling
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C
- Sterne
- 40.4k
- Forks
- 8.2k
- Ø Merge
- 2 T. 13 Std.
- Gemergte PRs (30 T.)
- 96
Beschreibung
Description
This is in relation to https://bugs.php.net/bug.php?id=73149 (with 142 up votes) which points problematic error handling of dns_get_record.
Due to BC break, this cannot be handled as a bug fix but there is a space to improve the error handling.
Firstly it would be useful to introduce parameter to switch the error handling to throw exception (e.g. bool $throw), that would allow throwing specialized exception which would allow to identify type of the error.
Additionally it might be also useful to add retry mechanism as some failures for TRY_AGAIN error.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die Implementierungen und Tests für dns_get_record und dns_check_record in php-src zu finden. Lies den verlinkten Fehlerbericht zu den bestehenden Anforderungen an die Fehlerbehandlung und kläre den vorgeschlagenen Ausnahmeparameter, die spezialisierten Fehlertypen und das Wiederholungsverhalten, bevor du den Code änderst. Die Aufgabe ist erledigt, wenn das vereinbarte API-Verhalten dokumentiert und durch Tests abgedeckt ist, ohne bestehende Aufrufer zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- c, php
- Bereich
- backend-api-design, networking
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100