Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Range is not validated #1

Open
razvan-z opened this issue Nov 1, 2018 · 1 comment
Open

Range is not validated #1

razvan-z opened this issue Nov 1, 2018 · 1 comment
Labels
bug

Comments

@razvan-z
Copy link

razvan-z commented Nov 1, 2018

Date range is not validated.
For example, 2018-11-31T09:00:00+00:00 passes the test, but YYYY-11-31 is not a valid date (November has 30 days).
@penance316
Copy link
Owner

Thanks for the spot.
I think the regex used here does not cover the ISO8601 spec 100% for example time interval and duration will not match but I agree it would be nice if the actual date was validated.
Will see if we can get the regex tidied up a bit

@penance316 penance316 added the bug label Nov 9, 2018
penance316 pushed a commit that referenced this issue Jan 2, 2019
@pablorsk
Feature/utc parameter (#1)
49ff5aa 
* utc parameter to rule
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@penance316 @razvan-z