fail harder if we can't write announcement-seqnum #2905

Open
opened 2018-02-21 00:28:00 +00:00 by meejah · 0 comments
Owner

If the file announcement-seqnum in the node dir isn't writable, the code in write_config (in Node, or in new _Config class) will catch and log the error.

However, this isn't very noticeable and we should probably fail harder (e.g. like exiting with an error). Other things which write config-files into the node dir should also be examined.

In this particular case, if announcement-seqnum doesn't get written (and the user doesn't notice the error in the logs) then the next time we try and make an announcement (next re-start) the introducer will reject it.

If the file `announcement-seqnum` in the node dir isn't writable, the code in `write_config` (in `Node`, or in new `_Config` class) will catch and log the error. However, this isn't very noticeable and we should probably fail harder (e.g. like exiting with an error). Other things which write config-files into the node dir should also be examined. In this particular case, if announcement-seqnum *doesn't* get written (and the user doesn't notice the error in the logs) then the next time we try and make an announcement (next re-start) the introducer will reject it.
meejah added the
code-storage
normal
defect
1.12.1
labels 2018-02-21 00:28:00 +00:00
meejah added this to the undecided milestone 2018-02-21 00:28:00 +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.

Reference: tahoe-lafs/trac-2024-07-25#2905
No description provided.