Skip to content

Conversation

@sirtimid
Copy link
Contributor

Closes #52

When a vat is deleted inside the IframeManager, any unresolved messages sent to the vat return an error to the original sender and then we drop all references to those messages.

@sirtimid sirtimid requested a review from a team as a code owner September 13, 2024 17:40
@sirtimid sirtimid changed the title fix: Reject unresolved messages instead of warning feat(extension): Handle orphaned messages on vat deletion Sep 13, 2024
@rekmarks rekmarks self-assigned this Sep 13, 2024
@rekmarks rekmarks self-requested a review September 13, 2024 17:45
@rekmarks rekmarks removed their assignment Sep 13, 2024
Copy link
Member

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sirtimid sirtimid merged commit d9a1741 into main Sep 13, 2024
@sirtimid sirtimid deleted the sirtimid/handle-orphaned-messages branch September 13, 2024 17:56
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

Successfully merging this pull request may close these issues.

feat(extension): Handle orphaned messages on vat deletion

3 participants