validatorjs/validator.js

isISO8601 returns true for an invalid iso8601 date

Open

#2.003 geöffnet am 18. Juli 2022

Auf GitHub ansehen
 (16 Kommentare) (1 Reaktion) (1 zugewiesene Person)JavaScript (22.497 Stars) (2.253 Forks)batch import
good first issue🐛 bug

Beschreibung

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

Contributor Guide

Tech Stack
javascript
Domain
backend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
blocked
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaScriptRegular expressionsValidator.js structure
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65
Research-Richtung
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.