Project

General

Profile

Task #2892

Deprecate unnecessary Pib/Tpm specific methods from KeyChain

Added by Yingdi Yu over 3 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Security
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:

Description

As KeyChain used to inherit from Pib and Tpm, many KeyChain methods are there simply because they are the methods of Pib & Tpm.
These methods are not semantically methods of KeyChain, and increase the complexity of KeyChain interface, and should be removed from KeyChain interfaces.

Current KeyChain interfaces provide four categories of methods:

  1. add/delete create identity, key, and certificate
  2. packet signing
  3. lookup identity, key, and certificate
  4. some Pib or Tpm specific methods

The first two categories have KeyChain semantic and require both Pib and Tpm, and should be kept.
The third category only requires Pib, and can be done easily through the existing method KeyChain::getPib(), and should be removed.
The last category is only meaningful to Pib and Tpm, and should not be exposed as KeyChain methods.

The plan is to first deprecate the last two category methods in v0.4, and remove them in v0.5.


Related issues

Blocks ndn-cxx - Task #2890: Delete deprecated KeyChain methodsAbandoned

Blocks ndn-cxx - Task #2926: Refactor KeyChainClosed

History

#1 Updated by Yingdi Yu over 3 years ago

  • Target version set to v0.4

#2 Updated by Junxiao Shi over 3 years ago

  • Description updated (diff)

#3 Updated by Junxiao Shi over 3 years ago

  • Blocks Task #2890: Delete deprecated KeyChain methods added

#4 Updated by Yingdi Yu over 3 years ago

#5 Updated by Junxiao Shi over 3 years ago

  • Subject changed from Deprecate unnecessary method from KeyChain to Deprecate unnecessary Pib/Tpm specific methods from KeyChain
  • Description updated (diff)
  • Start date deleted (06/12/2015)

#6 Updated by Yingdi Yu over 2 years ago

  • Status changed from New to Code review
  • % Done changed from 0 to 80

#7 Updated by Yingdi Yu over 2 years ago

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

Also available in: Atom PDF