DigitalTrustCenter / sectxt

security.txt parser and validator
European Union Public License 1.2
17 stars 6 forks source link

Fix #66 - Fix validation for leap days #67

Closed mxsasha closed 7 months ago

DigitalTrustCenter commented 7 months ago

Leap years did seem to fail since it just added a 1 to the year value without accounting for these leap days.

While your solution would solve the breaking issue with the value error it would not be entirely correct, since it would now count a day short during a leap year.

Your fix has been altered slightly to fix the value error and account for the leap days and it will be added in version 0.9.2. Thank you for reporting the issue and for the offered solution.