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

The "0 0 0 ? * Mon/2" is an invalid cron string with quartz 3.0.1 #573

Closed
guotuofeng opened this issue Jan 22, 2018 · 3 comments
Closed

Comments

@guotuofeng
Copy link

guotuofeng commented Jan 22, 2018

Quartz.net 3.0.1

Expected behavior

The "0 0 0 ? * Mon/2" should be a valid cron string which should be triggered every two weeks on Monday.

In Quartz 2.5, it works fine.

Actual behavior

The exception is thrown

System.FormatException: 'Illegal characters for this position: 'MON''

Steps to reproduce

            var cron_expr = "0 0 0 ? * Mon/2";
            CronExpression.IsValidExpression(cron_expr); // should be true, but return false
@guotuofeng guotuofeng changed the title The " "0 0 0 ? * Mon/2" is an valid cron string with quartz 3.0.1 The " "0 0 0 ? * Mon/2" is an invalid cron string with quartz 3.0.1 Jan 22, 2018
@guotuofeng guotuofeng changed the title The " "0 0 0 ? * Mon/2" is an invalid cron string with quartz 3.0.1 The "0 0 0 ? * Mon/2" is an invalid cron string with quartz 3.0.1 Jan 23, 2018
@lahma
Copy link
Member

lahma commented Jul 6, 2018

I believe this did parse as valid in 2.x but did never obey the every second week rule. This is why the validation now throws error as it's not supported.

@toolgood
Copy link

This error also exists in 3.06.

@lahma
Copy link
Member

lahma commented Jul 13, 2020

I believe this was fixed with #790 and already part of 3.1 beta 1.

@lahma lahma closed this as completed Jul 13, 2020
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

3 participants