-
Notifications
You must be signed in to change notification settings - Fork 102
Add change notification expectations to Security Considerations. #916
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
TallTed
left a comment
There was a problem hiding this 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>
| [=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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| and be a core part of change notification service specifications. | |
| and be a core part of the specifications of change notification services. |
msporny
left a comment
There was a problem hiding this 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.
|
This was discussed during the #did meeting on 15 January 2026. View the transcriptw3c/did#916ottomorac: 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 PR is an attempt to address issue #909 by adding add change notification service expectations to the Security Considerations section.
Preview | Diff