Fix Delete Profile Flow #560

Closed
opened 2022-10-24 16:29:50 +00:00 by sarah · 1 comment
Owner

Seems like a fallout from the removal of appclient / API needs better error handling.

Ref: cwtch.im/cwtch#473

Seems like a fallout from the removal of appclient / API needs better error handling. Ref: https://git.openprivacy.ca/cwtch.im/cwtch/issues/473
sarah added the
bug
cwtch-beta-1.10
bugbash
labels 2022-10-24 16:29:50 +00:00
sarah added this to the Cwtch Beta (In Progress) project 2022-10-24 16:33:19 +00:00
Author
Owner

This is actually 2 bugs:

  1. shutdown is called before delete on peer, and because shutdown purges peer maps it causes a NPE
  2. check password is incomplete in peer and needs to also include a check that the underlying database is valid cwtch storage (this only impacts delete peer, as change password was doing the right check) - need to consolidate this logic.

Will integrate these changes into my upcoming Cwtch PR

This is actually 2 bugs: 1. shutdown is called before delete on peer, and because shutdown purges peer maps it causes a NPE 2. check password is incomplete in peer and needs to also include a check that the underlying database is valid cwtch storage (this only impacts delete peer, as change password was doing the right check) - need to consolidate this logic. Will integrate these changes into my upcoming Cwtch PR
sarah self-assigned this 2022-11-23 16:20:05 +00:00
sarah added the
in-progress
label 2022-11-24 22:02:12 +00:00
sarah referenced this issue from a commit 2022-11-30 19:42:41 +00:00
dan closed this issue 2022-11-30 19:43:55 +00:00
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cwtch.im/cwtch-ui#560
No description provided.