Skip to content

docs: document Header Confidentiality Policy#8016

Merged
link2xt merged 1 commit intomainfrom
link2xt/owwmrosxyllr
Mar 23, 2026
Merged

docs: document Header Confidentiality Policy#8016
link2xt merged 1 commit intomainfrom
link2xt/owwmrosxyllr

Conversation

@link2xt
Copy link
Collaborator

@link2xt link2xt commented Mar 22, 2026

@link2xt link2xt force-pushed the link2xt/owwmrosxyllr branch from 09ec7a1 to 1eeeeb9 Compare March 22, 2026 05:03
@link2xt link2xt marked this pull request as ready for review March 22, 2026 05:04
@link2xt link2xt force-pushed the link2xt/owwmrosxyllr branch 2 times, most recently from beed0f1 to fb2f732 Compare March 22, 2026 05:32
r10s
r10s approved these changes Mar 22, 2026
View reviewed changes
Copy link
Contributor

@r10s r10s left a comment

Choose a reason for hiding this comment

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

thanks for taking care!

lgtm, but i am wondering "if if val_in is an RFC 5322 date-time" is needed in this spec, regarding level of detail. it seems just to be a technical detail in case core sets date wrong elsewhere. similar for "from"

@link2xt link2xt force-pushed the link2xt/owwmrosxyllr branch from fb2f732 to 39e8dd8 Compare March 22, 2026 06:15
@link2xt
Copy link
Collaborator Author

link2xt commented Mar 22, 2026

lgtm, but i am wondering "if if val_in is an RFC 5322 date-time" is needed in this spec, regarding level of detail. it seems just to be a technical detail in case core sets date wrong elsewhere. similar for "from"

I removed if val_in is an RFC 5322 date-time, it is not needed because it does not matter what original Date is. For the From, it cannot happen in the core, but HCP is written for email clients where the whole mail with headers may be modified by the user.

I also don't really understand the idea of registering HCP that RFC has, HCP is actually an implementation detail, so it does not need naming and registering.

link2xt
link2xt commented Mar 22, 2026
View reviewed changes
spec.md Outdated
return the UTC form of a random date within the last 7 days
else if lower(name) is 'subject':
return '[...]'
else if lower(name) is in ['message-id', 'chat-version', 'chat-is-post-message']:
Copy link
Collaborator Author

@link2xt link2xt Mar 22, 2026

Choose a reason for hiding this comment

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

We want to remove this chat-version from outer headers eventually. Maybe should do it now in the doc already. We don't need it and only send it for compatibility: #7932

Copy link
Contributor

@r10s r10s Mar 22, 2026

Choose a reason for hiding this comment

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

makes sense

@link2xt link2xt force-pushed the link2xt/owwmrosxyllr branch from 39e8dd8 to 971f239 Compare March 23, 2026 01:07
@link2xt link2xt merged commit aedc60f into main Mar 23, 2026
29 of 30 checks passed
@link2xt link2xt deleted the link2xt/owwmrosxyllr branch March 23, 2026 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@r10s r10s r10s approved these changes

@iequidoo iequidoo iequidoo approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@link2xt @r10s @iequidoo