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.

Sign up for GitHub

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

Jump to bottom

Version bump and fix chrono timestamp_opt warn #156

Open

thespooler wants to merge 2 commits into HeroicKatora:master from thespooler:fix_timestamp_warn

Contributor

thespooler commented Dec 3, 2022 •

edited

This fixes timestamp deprecation warning by moving to timestamp_opt() as the behavior of timestamp() is to panic on error.

I have read the contribution guidelines
This change has tests (remove for doc only)
This change has documentation
Corresponds to issue (number)


          Version bump and fix chrono timestamp_opt warn

9dab318

HeroicKatora reviewed

View reviewed changes

oxide-auth/src/primitives/generator.rs Outdated Show resolved Hide resolved


          More descriptive error on timestamp conversion

bbd5566

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment