Skip to content

Add change notification expectations to Security Considerations.#916

Merged
msporny merged 3 commits intomainfrom
msporny-sing-change-notifications
Feb 21, 2026
Merged

Add change notification expectations to Security Considerations.#916
msporny merged 3 commits intomainfrom
msporny-sing-change-notifications

Conversation

@msporny
Copy link
Copy Markdown
Member

@msporny msporny commented Jan 12, 2026

This PR is an attempt to address issue #909 by adding add change notification service expectations to the Security Considerations section.


Preview | Diff

@msporny msporny self-assigned this Jan 12, 2026
@msporny msporny marked this pull request as ready for review January 12, 2026 16:55
@msporny msporny added the class 2 Changes that do not functionally affect interpretation of the document label Jan 13, 2026
Comment thread index.html Outdated
Comment thread index.html Outdated
Copy link
Copy Markdown
Member

@TallTed TallTed left a comment

Choose a reason for hiding this comment

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

A few small changes suggested. Approved either way.

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
Comment thread index.html Outdated
[=DID document=], and rate limits are expected to be documented and be a core
part of change notification service specifications.
[=DID document=], and enforcement of rate limits are expected to be documented
and be a core part of change notification service specifications.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
and be a core part of change notification service specifications.
and be a core part of the specifications of change notification services.

Copy link
Copy Markdown
Member Author

@msporny msporny left a comment

Choose a reason for hiding this comment

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

The group discussed this on 2026-01-15 and decided that we should remove the section because we haven't done an analysis on change notification sections and don't have much implementation experience and it is confusing/concerning people that are reading the section.

The PR should be modified to remove the section.

@w3cbot
Copy link
Copy Markdown

w3cbot commented Jan 15, 2026

This was discussed during the #did meeting on 15 January 2026.

View the transcript

w3c/did#916

ottomorac: Manu can you give us a summary

manu: This one is a request from PING to detail considerations for change notification text

manu: this is about VDRs, with a DID in it, how do you let people know about that

manu: PING thinks this matters. Talk about what you need to do to harden that surface

<Zakim> JoeAndrieu, you wanted to ask why we have change notification

JoeAndrieu: I think we should get this section out of the spec...

<swcurran> +1 to Joe

JoeAndrieu: I think we should not be speaking to this in spec

manu: I'm fine with doing that as well. This text came from the early days.
… This talked as if this is a regular thing, but it isn't really.
… It confused the reviewers. Since we haven't done the analysis, we should probably stay quiet


Copy link
Copy Markdown

@mccown mccown left a comment

Choose a reason for hiding this comment

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

Approve updates.

@msporny
Copy link
Copy Markdown
Member Author

msporny commented Feb 21, 2026

… It confused the reviewers. Since we haven't done the analysis, we should probably stay quiet

I have removed the section, per the discussion in the DID WG.

@msporny
Copy link
Copy Markdown
Member Author

msporny commented Feb 21, 2026

Editorial, multiple reviews, changes requested and made, no objections, merging.

@msporny msporny merged commit a0a2918 into main Feb 21, 2026
2 checks passed
@msporny msporny deleted the msporny-sing-change-notifications branch February 21, 2026 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

class 2 Changes that do not functionally affect interpretation of the document

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants