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

Add new status update message type #161

Open
Patrik-Stas opened this issue Apr 8, 2022 · 0 comments
Open

Add new status update message type #161

Patrik-Stas opened this issue Apr 8, 2022 · 0 comments

Comments

@Patrik-Stas
Copy link
Contributor

Patrik-Stas commented Apr 8, 2022

  • Message type MSGTYPE_UPDATE_MSG_STATUS_BY_CONNS has following format
{
    "@type": <MSGTYPE_UPDATE_MSG_STATUS_BY_CONNS>,
    "statusCode": "MS-106",
    "uidsByConns": [{"pairwiseDID":"6FRuB95abcmzz1nURoHyWE","uids":["Br4CoNP4TU"]}, ...]
}
  • In PR Optimize message status updates #159 we changed behaviour of MSGTYPE_UPDATE_MSG_STATUS_BY_CONNS such that values pairwiseDID are ignored and simply merge of uid is performed, whereas we guarantee on DB level that uids are unique across all agent connections.

We should deprecate MSGTYPE_UPDATE_MSG_STATUS_BY_CONNS and replace it by new message type with simpler argument interface.
Proposal

{
    "@type": <MSGTYPE_UPDATE_MSG_STATUS_BY_UID>,
    "statusCode": "MS-106",
   "uids":["Br4CoNP4TU", ... ] 
}
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

1 participant