validatorjs/validator.js

isISO8601 returns true for an invalid iso8601 date

Ouverte

#2 003 ouverte le 18 juil. 2022

Voir sur GitHubGuide contributeur
 (16 commentaires) (1 réaction) (1 personne assignée)JavaScript (2 489 forks)batch import
good first issue🐛 bug

Métriques du dépôt

Stars
 (23 739 étoiles)
Métriques de merge PR
 (Merge moyen 10h 14m) (2 PRs mergées en 30 j)

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

Direction de recherche
Examinez la fonction isISO8601 dans le code source du validateur, identifiez le motif regex qui accepte actuellement les dates sans secondes (comme '2018-09-11T10:16.000Z'), et corrigez le pour exiger des secondes ou traiter l'absence de secondes comme invalide.
Stack technique
javascript
Domaine
api
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptRegular Expressions
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65