Project

General

Profile

Actions
Copy link

Task #2920

closed

Replace SecPublicInfo with Pib

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

Status:
Abandoned
Priority:
Normal
Assignee:
-
Category:
Security
Target version:
-
Start date:
06/18/2015
Due date:
% Done:

0%

Estimated time:

Description

With PibSqlite3 is implemented, we can now replace SecPublicInfo with Pib.

We need a KeyChain method to return reference of Pib. However, Current KeyChain::getPib() returns a SecPublicInfo reference. We propose to use KeyChain::Pib() for now, and then deprecate KeyChain::getPib(), and then change KeyChain::getPib() to return Pib ref

Actions
Copy link
 #1

Updated by Yingdi Yu almost 9 years ago

I will try ndn-cxx-breaks first to see the impact of changing return type.

Actions
Copy link
 #2

Updated by Junxiao Shi almost 9 years ago

What would be the behavior of the SecPublicInfo object returned from KeyChain::getPib method? There shouldn't be two concurrent PIBs.

Is it a wrapper over the Pib object?

Actions
Copy link
 #3

Updated by Yingdi Yu almost 9 years ago

Junxiao Shi wrote:

What would be the behavior of the SecPublicInfo object returned from KeyChain::getPib method? There shouldn't be two concurrent PIBs.

No, therefore, it will throw exception...

Is it a wrapper over the Pib object?

No...

Actions
Copy link
 #4

Updated by Junxiao Shi almost 9 years ago

Since SecPublicInfo is no longer working, this change is backwards-incompatible.
I suggest putting it into the feature branch together with #2306 and perhaps Validator changes, so that applications only need to change once.

Actions
Copy link
 #5

Updated by Alex Afanasyev over 7 years ago

  • Status changed from New to Abandoned

This semi-not applicable, semi-finished. The v2::KeyChain is using Pib interface, v1::KeyChain that use SecPublicInfo is deprecated and will be removed soon.

Actions
Copy link

Also available in: Atom PDF