Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 564819 - [Passage] API revision | conditions | stream codecs review
Summary: [Passage] API revision | conditions | stream codecs review
Status: CLOSED FIXED
Alias: None
Product: Passage
Classification: Technology
Component: API (show other bugs)
Version: 0.9.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.0.0   Edit
Assignee: Elena Parovyshnaia CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 564419
  Show dependency tree
 
Reported: 2020-07-01 04:03 EDT by Elena Parovyshnaia CLA
Modified: 2020-07-03 09:30 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Elena Parovyshnaia CLA 2020-07-01 04:03:45 EDT 
In the scope of general API revision StreamCodec-related interfaces and implementations are to be also questioned against new API approach guides, revised and, most probably, reimplemented. 

As the result
  - new interfaces and implementation must appear complying with the new API approach. Functionality must be kept.
  - old ones are to be deprecated
  - tests should appear where possible
Comment 1 Elena Parovyshnaia CLA 2020-07-02 13:30:39 EDT 
PR#1 https://github.com/eclipse-passage/passage/pull/275
   - invite NamedData-based coverage for public string literals (encryption algorithm and key size)

PR#2 https://github.com/eclipse-passage/passage/pull/277
 - redesign StreamCodec and registry interfaces
 - rework existing BcStreamCodec implementation
 - deprecate it
 - plug the new one to Demo Framework
Comment 2 Elena Parovyshnaia CLA 2020-07-03 08:46:59 EDT 
PR#3 https://github.com/eclipse-passage/passage/pull/278
   - exhaustively test key pair generation
   - fix BcStreamCodec according to the test expectation

PR#4 https://github.com/eclipse-passage/passage/pull/279
  - test encoding for new BcStreamCodec

PR#5 https://github.com/eclipse-passage/passage/pull/280
  - test decoding for new BcStreamCodec
Comment 3 Elena Parovyshnaia CLA 2020-07-03 09:30:04 EDT 
`lic.api` / `lic.internal.api.io` - general interfaces for codec, registry and all the required configurational constructions, documented

`lic.base` / `lic.internal.base.io` - basic implementations for satellite structures

`lic.bc` / `lic.internal.bc` - reimpementation for BouncyCastle-based stream coding and decoding for contemporary interfaces

`lic.bc.tests` / `lic.internal.bc.tests` - tests set to be used as a cookbook, contract description and as documentation.