[go] time: fix time zone parsing when format includes time zone seconds

1 view
Skip to first unread message

Gopher Robot (Gerrit)

unread,
Jul 2, 2024, 4:02:00 PM (3 days ago) Jul 2
to Darin Krauss, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Than McIntosh, Go LUCI, Ian Lance Taylor, Russ Cox, golang-co...@googlegroups.com

Gopher Robot submitted the change

Change information

Commit message:
time: fix time zone parsing when format includes time zone seconds

The current implementation fails to parse a time string with a "Z"
time zone using a time format that includes time zone seconds. This
fix correctly parses the "Z" time zone for any Z-base time format
that includes seconds (i.e. "Z070000" or "Z07:00:00").

Fixes #68263
Change-Id: Idf8fa06b5f96383f050c4ffbd2bc5804fd408650
Auto-Submit: Ian Lance Taylor <ia...@google.com>
Reviewed-by: Ian Lance Taylor <ia...@google.com>
Reviewed-by: Than McIntosh <th...@google.com>
Files:
  • M src/time/format.go
  • M src/time/format_test.go
Change size: S
Delta: 2 files changed, 21 insertions(+), 2 deletions(-)
Branch: refs/heads/master
Submit Requirements:
  • requirement satisfiedCode-Review: +2 by Ian Lance Taylor, +1 by Than McIntosh
  • requirement satisfiedTryBots-Pass: LUCI-TryBot-Result+1 by Go LUCI
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Idf8fa06b5f96383f050c4ffbd2bc5804fd408650
Gerrit-Change-Number: 595897
Gerrit-PatchSet: 2
Gerrit-Owner: Darin Krauss <darin...@gmail.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Ian Lance Taylor <ia...@google.com>
Gerrit-Reviewer: Russ Cox <r...@golang.org>
Gerrit-Reviewer: Than McIntosh <th...@google.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages