Project

General

Profile

Actions

Bug #3351

closed

Inconsistent behavior of PibImpl implementation

Added by Yingdi Yu almost 9 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Security
Target version:
Start date:
11/23/2015
Due date:
% Done:

100%

Estimated time:

Description

We defined PibImpl interface and had two PibImpl implementations: PibSqlite3 and PibMemory.
However the behavior of these two are not consistent with each other regarding adding key and certificate.
For example, PibMemory will overwrite existing record when it shares the same name as the added one, while PibSqlite3 will do nothing.

In the current document of these methods, I put "if the key/cert exists, do nothing". However, I do not consider this as good behavior now. We either throw Pib::Error, or update the record and leave existence checking to the caller. I prefer to do update directly to avoid throwing exceptions, and state clearly in document about this behavior.

The fix to this problem involves fixing the document and the implementation.

Actions #1

Updated by Yingdi Yu almost 9 years ago

  • Status changed from New to Code review
Actions #2

Updated by Alex Afanasyev almost 9 years ago

  • Target version set to v0.5
Actions #3

Updated by Yingdi Yu over 8 years ago

  • Status changed from Code review to Closed
  • % Done changed from 0 to 100

addressed in a reviewed commit http://gerrit.named-data.net/#/c/2604/

Actions

Also available in: Atom PDF