bugmonotone - Bugs: bug #12221, unify bad_decode() and error()


You are not allowed to post comments on this tracker with your current authentication level.

bug #12221: unify bad_decode() and error()

Submitted by:  Nathaniel Smith <njs>
Submitted on:  Sat 05 Mar 2005 10:44:10 AM UTC  
Category: networkingSeverity: 1 - Wish
Item Group: code cleanupStatus: None
Privacy: PublicAssigned to: None
Open/Closed: Open
mtn version --full: 

Sat 05 Mar 2005 10:44:10 AM UTC, original submission:

we have two different ways of reporting errors in netsync. the advantage of bad_decode is that it triggers a transaction rollback. the advantage of error is that it informs the peer of the error before shutting down the connection.

these should be merged into a single error-reporting functionality, so that all errors inform the peer of the problem (a significant usability improvement), and all errors trigger a database rollback.

beware of 2nd-level errors encountered while informing the peer of an error -- these must be handled correctly as well (presumably by giving up on telling the peer anything useful and just shutting down the connection immediately).

Nathaniel Smith <njs>
Project Member


No files currently attached


Depends on the following items: None found

Items that depend on this one: None found


CC list is empty


Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.


Please enter the title of George Orwell's famous dystopian book (it's a date):



No Changes Have Been Made to This Item

Back to the top

Powered by Savane 3.1-cleanup1