php / php/php-src

Improve dns_get_record / dns_check_record error handling

Abierto
#17,919 1 comentario 4 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Extension: standard Feature
Lenguaje dominante
C
Estrellas
40.4k
Forks
8.2k
Merge medio
2 d 13 h
PR fusionados (30 d)
96

Descripción

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.

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza localizando las implementaciones y las pruebas de dns_get_record y dns_check_record en php-src. Lee el informe de errores enlazado sobre los requisitos existentes de manejo de errores y aclara el parámetro de excepción propuesto, los tipos de error especializados y el comportamiento de reintento antes de cambiar el código. El trabajo estará terminado cuando el comportamiento acordado de la API esté documentado y cubierto por pruebas sin romper a los llamadores existentes.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
c, php
Área
backend-api-design, networking
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.