Skip to content
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

Discuss/Define: Notifications #24

Closed
jischr opened this issue Dec 16, 2024 · 6 comments
Closed

Discuss/Define: Notifications #24

jischr opened this issue Dec 16, 2024 · 6 comments

Comments

@jischr
Copy link

jischr commented Dec 16, 2024

topics to cover:

  • Receive Notifications of Revoked Tokens
    • be notified when tokens have been revoked
  • Receive Notifications of Invalidated Sessions
    • be notified when sessions have been invalidated
@dhs-BI
Copy link
Contributor

dhs-BI commented Dec 17, 2024

At first glance, I didn't understand why you didn't include #25 in this list, now I see you've separated the items.

Is there a reason these are defined as "notifications" and #25 is defined as "signals?"

@jischr
Copy link
Author

jischr commented Dec 17, 2024

@dhs-BI i just broke down based on the sections in https://github.com/openid/ipsie/blob/main/ipsie-v1-draft.md#receive-notifications-of-invalidated-sessions

i was curious too if there was a difference semantically or functionally between "notifications" and "signals"

@aaronpk
Copy link
Collaborator

aaronpk commented Dec 17, 2024

I think those two are meant to be the same. The point of both "be notified when sessions have been invalidated" and "receive real-time signals about changes in account posture or integrity" is that the sender says "this happened", with no expectation of the behavior of the recipient. This is mean to be in contrast to something like backchannel logout which is a "command" that has a clear expectation of what the recipient should do.

@dhs-BI
Copy link
Contributor

dhs-BI commented Dec 17, 2024

Thanks @aaronpk and @jischr. Let's discuss on the call today with the intent of merging these into a single set of items.

@timcappalli
Copy link
Member

2025-01-07 call: notifications tend to be user centric. Group leaning towards "signals" (#25).

@aaronpk
Copy link
Collaborator

aaronpk commented Jan 10, 2025

I moved these two over to #25.

@aaronpk aaronpk closed this as completed Jan 10, 2025
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

No branches or pull requests

4 participants