Improve dns_get_record / dns_check_record error handling
Personne n'a encore pris cette issue.
- Langage dominant
- C
- Étoiles
- 40.4k
- Forks
- 8.1k
- Merge moyen
- 2 j 13 h
- PR mergées (30 j)
- 96
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par localiser les implémentations et les tests de dns_get_record et dns_check_record dans php-src. Lisez le rapport de bug lié concernant les exigences existantes de gestion des erreurs et clarifiez le paramètre d’exception proposé, les types d’erreur spécialisés et le comportement de nouvelle tentative avant de modifier le code. Le travail est terminé lorsque le comportement convenu de l’API est documenté et couvert par des tests sans casser les appelants existants.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- c, php
- Domaine
- backend-api-design, networking
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100