validatorjs/validator.js

isISO8601 returns true for an invalid iso8601 date

Open

#2 003 ouverte le 18 juil. 2022

Voir sur GitHub
 (16 commentaires) (1 réaction) (1 assigné)JavaScript (22 497 stars) (2 253 forks)batch import
good first issue🐛 bug

Description

Describe the bug When using isISO8601 to check date that looks like iso8601 format, but missing seconds, will return true even though it is an invalid date

Examples Date string such as "2018-09-11T10:16.000Z" will return true. Live sample: https://codesandbox.io/s/wandering-fire-qr6fm6?file=/src/index.js

Additional context Validator.js version: 13.7.0 Node.js version: 16 OS platform: macOS

Guide contributeur

Stack technique
javascript
Domaine
backend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
blocked
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaScriptRegular expressionsValidator.js structure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65
Direction de recherche
Examine the isISO8601 function implementation in the codebase (likely src/lib/isISO8601.js). The bug is that the current regex accepts dates without seconds (e.g., '2018-09-11T10:16.000Z'). Review the issue comments for proposed regex patterns or discussions. Update the regex to require seconds or adjust the validation logic accordingly. Ensure existing tests pass and add new tests for the invalid cases.