search

ietf-wg-gnap / gnap-core-protocol

143 stars 26 forks source link

Added state diagram and discussion #427

Closed jricher closed 2 years ago

jricher commented 2 years ago

Adds a diagram and discussion of protocol state.

Closes #353 Closes #95 Closes #94 Closes #93 Closes #92 Closes #90

netlify[bot] commented 2 years ago

Deploy Preview for gnap-core-protocol-editors-draft ready!

Name Link
Latest commit 6526174269fa61643b61dab1f5f084825e7e6e97
Latest deploy log https://app.netlify.com/sites/gnap-core-protocol-editors-draft/deploys/62c5ad90519e6400090da6a2
Deploy Preview https://deploy-preview-427--gnap-core-protocol-editors-draft.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

yaronf commented 2 years ago

The text (but probably not the diagram) should mention that each state can also transition into Finalized if a timeout occurs, e.g. if the client is expected to send a continuation request but doesn't do so.

jricher commented 2 years ago

I think the current state text does mention the timeout in each state, but I did go back and add some more explicit explanation to #428 that was lacking.