Have a hard stop in Cron#next_time (and #previous_time)

[floraison]

Have a hard stop in Cron#next_time (and #previous_time)

[jmettraux]

Included in the 1.1.7 release. Many thanks.

[jmettraux]

Duplicate of #10, I should release ASAP.

[jmettraux]

@godfat Would that suit you if I released fugit 1.1.7 immediately?

[godfat]

@jmettraux Thank you very much for the quick reactions! I didn't have time to look into how this was implemented yet, but my main concern is that next_time can potentially never stop. This is way more harmful than accepting an invalid cron because it's very easily to shut down a service with that (i.e. a DoS attack). I really hope the implementation can be done in a way that it's impossible to get into this situation.

We're not using fugit yet, so please take your time, and there's no need to release a new version immediately for us, but I think that's indeed a very critical fix which should be available to other users soon.

On the other hand, I will be hesitant to make the switch before I am sure fugit will always stop in a timely manner. I'll verify the implementation later. (e.g. a loop has a hard limit to stop and raise an error)

Thanks again!

[jmettraux]

@godfat

We are in sync, I was thinking about such a hard stop as I was waiting for the bus. I will implement it.

Best regards and thanks again.

[godfat]

Awesome, thank you!