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

Add support for RFC822 timezone formatting for HeaderDateParser. #12

Merged
merged 1 commit into from
Jun 14, 2021

Conversation

JohnTasler
Copy link
Contributor

@JohnTasler JohnTasler commented Jan 8, 2020

While trying out the API against a newsgroup server I use, I ran into various header timezone strings that were failing to parse. I updated the timezone parsing to handle all cases specified in RFC822, as well as "UTC", which is not specified in the RFC but is a common timezone string nonetheless.

Additionally, I ran into a header date string that was suffixed with " (UTC)". Since this is another token, I updated the token-count validation logic for this specific case.

@JohnTasler
Copy link
Contributor Author

@keimpema Please take a look at the AppVeyor build failure. It appears to be related to infrastructure rather than the code change.

@keimpema keimpema merged commit ac2bda4 into keimpema:master Jun 14, 2021
@JohnTasler JohnTasler deleted the jt branch November 13, 2021 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants