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.

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

We need more validation for step #478

Open
peterqiu07 opened this issue Jan 16, 2023 · 0 comments
Open

We need more validation for step #478

peterqiu07 opened this issue Jan 16, 2023 · 0 comments

Comments

@peterqiu07
Copy link

cron/parser.go

Line 313 in bc59245

if step == 0 {

There is not enough validation for step
So when we use cron expression like "0/9999 * * * * ?", it still works, but it seems like out of expectation, and more strange
Further more, what do you expect for "0/9999 0/666 * * * ?"
image

Maybe we need to use the bounds to limit range as well as start and end

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant