Skip to content

Fix timezone formatting for RFC1123 on some Android devices.#288

Merged
jpd236 merged 1 commit intogoogle:masterfrom
jpd236:rfc1123-header
Aug 29, 2019
Merged

Fix timezone formatting for RFC1123 on some Android devices.#288
jpd236 merged 1 commit intogoogle:masterfrom
jpd236:rfc1123-header

Conversation

@jpd236
Copy link
Copy Markdown
Collaborator

@jpd236 jpd236 commented Aug 29, 2019

Some devices render a SimpleDateFormat string of 'zzz' as 'GMT+00:00'
instead of just 'GMT', which violates the specification. Since the
only valid timezone is 'GMT', just hardcode it as a constant to be
safe.

Fixes #287

Some devices render a SimpleDateFormat string of 'zzz' as 'GMT+00:00'
instead of just 'GMT', which violates the specification. Since the
only valid timezone is 'GMT', just hardcode it as a constant to be
safe. Don't change the parsing flow in case servers are returning
timezones other than 'GMT'.

Fixes google#287
@jpd236 jpd236 requested a review from jjoslin August 29, 2019 19:01
@jpd236 jpd236 merged commit b95b015 into google:master Aug 29, 2019
@jpd236 jpd236 deleted the rfc1123-header branch August 29, 2019 19:25
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.

Cache header If-Modified-Since does not conform to RFC1123

2 participants