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

Date header doesn't include timezone #20

Closed
d3zd3z opened this issue Oct 6, 2015 · 2 comments
Closed

Date header doesn't include timezone #20

d3zd3z opened this issue Oct 6, 2015 · 2 comments

Comments

@d3zd3z
Copy link

d3zd3z commented Oct 6, 2015

This may be a problem more with the 'time' crate, but the Tm::rfc822(), at least for me, doesn't include a timezone at the end, resulting in incorrect times displayed.

I have found it to work correctly if I replace the rfc822() in the code with rfc822z().

@amousset
Copy link
Member

amousset commented Oct 6, 2015

This is indeed a bug in the time crate, only UTC times are supported : https://github.com/rust-lang-deprecated/time/blob/master/src/display.rs#L243
I will switch to rfc822z() for now, thanks for reporting.

@amousset
Copy link
Member

amousset commented Oct 6, 2015

Fixed in 0.2.0 (5bedba4).

@amousset amousset closed this as completed Oct 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants