You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PR #31 fixes a sync issue but does so in a way that RecordsWrite overwrites that mutate one of the 5 mutable properties other than data are unintentionally processed by the DWN SDK's synchronizePrunedInitialRecordsWrite() method.
Issue created to track follow-through with one of the two following changes:
1. dwn-sdk-js change would be to modify the synchronizePrunedInitialRecordsWrite() method signature to take as input a RecordsWrite message with no associated data && a RecordsDelete.
2. dwn-server and web5-js modification to introduce a new endpoint / RPC handler that is dedicated to sync so that RecordsWrite overwrites that mutate properties other than data don't flow through synchronizePrunedInitialRecordsWrite() unintentionally.
The text was updated successfully, but these errors were encountered:
frankhinek
changed the title
Eliminate unintended pathway for record updates that uses synchronizePrunedInitialWrite
Eliminate unintended pathway for record updates that uses synchronizePrunedInitialRecordsWrite()Jun 1, 2023
PR #31 fixes a sync issue but does so in a way that
RecordsWrite
overwrites that mutate one of the 5 mutable properties other than data are unintentionally processed by the DWN SDK'ssynchronizePrunedInitialRecordsWrite()
method.Issue created to track follow-through with one of the two following changes:
1.
dwn-sdk-js
change would be to modify thesynchronizePrunedInitialRecordsWrite()
method signature to take as input aRecordsWrite
message with no associated data && aRecordsDelete
.2.
dwn-server
andweb5-js
modification to introduce a new endpoint / RPC handler that is dedicated to sync so thatRecordsWrite
overwrites that mutate properties other than data don't flow throughsynchronizePrunedInitialRecordsWrite()
unintentionally.The text was updated successfully, but these errors were encountered: