SecureIdentityAlliance / osia

Open Standard set of APIs for interoperability of identity management building blocks.
https://osia.readthedocs.io/en/latest/
Other
20 stars 20 forks source link

Notification : change part "confirm the address before the subscription is active" #34

Closed schesneau closed 3 years ago

schesneau commented 3 years ago
schesneau commented 3 years ago

Notification address confirmation :

olivier-heurtier-idemia commented 3 years ago

Hi @schesneau

This was indeed not very clear but your correction is not yet 100% correct.

Result should be:

N -> CR: subscribe_CB(message) activate N activate CR CR -> N: confirm(token) activate N #FFBBBB N --> CR: ok deactivate N CR --> N: ok deactivate CR deactivate N

This process is similar to the validation of an email when you register in a web site. You receive a first email with in the message a link (with a token) that you have to click to finalize your subscription.

The mechanism here is based on Amazon SNS which supports both direct HTTP calls for notification as well as email, SMS, others. This validation process is a little bit more complex to support those other media.

If it is OK for you, can you fix your PR so that it can be merged? Thanks

schesneau commented 3 years ago

Hi @olivier-heurtier-idemia

This way, it seem fine. It's ok for me.

Thanks.

coudot commented 3 years ago

Merge with @olivier-heurtier-idemia approval