Dutch (NL) postal code regex matches invalid values
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- internationalization
Línea de trabajo
Comienza con los datos de validación de códigos postales neerlandeses o el punto de entrada en libaddressinput y compara su regex con los patrones actuales y propuestos del issue. Revisa las pruebas unitarias de regex101 enlazadas en el informe y, a continuación, verifica que se rechacen los códigos con ceros iniciales y los sufijos SA, SD y SS, mientras que los códigos postales neerlandeses válidos sigan siendo aceptados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hello there,
Following a discussion at work, I looked into a package used for postal code validation (axlon/postal-code-validation), which is seeding data from the Google Address Data Service (as pointed out in axlon/postal-code-validation#27).
After a short discussion, @wotta pointed out that the regex used is invalid. The regex (\d{4} ?[A-Z]{2}) allows the following invalid postal codes:
0123 AB, which are invalid since all our postal codes start with 1-9.1111 SA,1111 SDand1111 SS, which are banned due to theSA,SD, andSSabbreviations which were used in the Second World War source.
A better regex for Dutch postal codes would be [1-9]\d{3} ?(?!SA|SD|SS)[A-Z]{2}, which explicitly excludes zero-based postal codes and invalid suffixes.
I made some tests on regex101.com (click on "Unit Tests" in the left sidebar).
- Lenguaje dominante
- C++
- Estrellas
- 631
- Forks
- 109
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de google/libaddressinput
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
google/libaddressinput#255 · 3 comentarios ·
-
Improve Spain postcode regexAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
google/libaddressinput#245 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 64/100
google/libaddressinput#254 ·
-
Error in Spain sub_zip listAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 52/100
google/libaddressinput#250 · 2 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 58/100
google/libaddressinput#249 ·
Todos los issues de google/libaddressinput
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
objectionary/eo-graphs#85 ·
-
bug derived types format I/O medium priority semantics
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
CartLineConstraint Jacobian always writes 6 rows, overruns when indices has fewer than 6 entriesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
tesseract-robotics/trajopt#618 ·
Los mantenedores suelen responder en 1 día
-
backend bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
mysql/mysql-server#847 ·
Los mantenedores suelen responder en 3 días