Skip to content

Clarify timestamps - #1410

Open
duglin wants to merge 1 commit into
cloudevents:mainfrom
duglin:timestamp
Open

duglin wants to merge 1 commit into
cloudevents:mainfrom
duglin:timestamp

Conversation

@duglin

@duglin duglin commented Nov 12, 2026 •

Copy link
Copy Markdown
Collaborator

@Cali0707 Cali0707 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (won't be able to attend meeting, but agree with this change)

Comment thread cloudevents/spec.md Outdated
- String encoding: [RFC 3339](https://tools.ietf.org/html/rfc3339).
- String encoding: [RFC 3339](https://tools.ietf.org/html/rfc3339). Note
that the RFC mentions using a character other that `T` as the separator
between the data and time; implementationts MUST use `T`. While lowercase

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
between the data and time; implementationts MUST use `T`. While lowercase
between the date and time; implementationts MUST use `T`. While lowercase

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doi - thanks!

Signed-off-by: Doug Davis <duglin@gmail.com>
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.

Timestamp parsing does not fully support RFC 3339

3 participants