aykevl / mta-sts

Online tool for MTA-STS checking: https://aykevl.nl/apps/mta-sts/
BSD 2-Clause "Simplified" License
39 stars 12 forks source link

Seconds in error message calculated incorrectly #9

Open paulmenzel opened 5 years ago

paulmenzel commented 5 years ago

posteo.de uses a small max_age value of five minutes.

max_age: 300

The service reports the warning below.

Error: Very short max_age field specified. It is less than a day with 0.003472222222222222 seconds. The suggested amount is at least in the order of weeks (this validator uses 28 days).

As the unit for max_age is integer seconds, shouldn’t it say 300 seconds in the message?

klausenbusk commented 3 years ago

PR opened: https://github.com/aykevl/mta-sts/pull/13