Project

General

Profile

Actions

Task #4124

closed

Add unit tests for security::transform::PublicKey encryption/decryption

Added by Zhiyi Zhang over 7 years ago. Updated about 7 years ago.

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

0%

Estimated time:
2.00 h

Description

Unit tests are needed for encryption/decryption function of security::transform::PublicKey.


Related issues 1 (0 open1 closed)

Related to ndn-cxx - Bug #4204: ndnsec: importing a previously exported safebag failsClosedLuca Keidel07/24/2017

Actions
Actions #1

Updated by Davide Pesavento over 7 years ago

  • Category set to Tests
  • Priority changed from Low to Normal
  • Start date deleted (06/08/2017)
Actions #2

Updated by Junxiao Shi over 7 years ago

  • Subject changed from No unit test for security::transform::PublicKey encryption/decryption function to Add unit tests for security::transform::PublicKey encryption/decryption
  • Description updated (diff)
  • Category changed from Tests to Security

In ndn-abac (Named Data Networking-Attribute Based Encryption), seems the encryption/decryption part is broken.

Exactly what is broken? Can you provide a snippet to reproduce?

Actions #3

Updated by Junxiao Shi over 7 years ago

  • Related to Bug #4204: ndnsec: importing a previously exported safebag fails added
Actions #4

Updated by Zhiyi Zhang over 7 years ago

I think I found the unit-test for the encryption/decryption. It's not in the unit test for the public key:
https://github.com/named-data/ndn-cxx/blob/master/tests/unit-tests/security/transform/private-key.t.cpp#L356

The unit test looks good. I will review my code in NAC-ABAC and NAC to see why things went wrong there.

Actions #5

Updated by Zhiyi Zhang over 7 years ago

  • Assignee set to Zhiyi Zhang
  • Priority changed from Normal to Low
Actions #6

Updated by Zhiyi Zhang about 7 years ago

  • Status changed from New to Closed

We actually have the encryption/decryption unit tests.

Actions

Also available in: Atom PDF