Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature: delegated management / token support #153

Closed
martinpaljak opened this issue Jan 26, 2019 · 1 comment
Closed

Feature: delegated management / token support #153

martinpaljak opened this issue Jan 26, 2019 · 1 comment

Comments

@martinpaljak
Copy link
Owner

Initial proposal is to add two options:
--token-key to generate the tokens on the fly
--token to add the token as byte array to a command
gregorjohannson pushed a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Feb 7, 2019
martinpaljak#153: Add initial functionality of including DM token cal…
f804da9 
…culation for INSTALL [for load]. Minor refactoring for more clarity.
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Feb 7, 2019
@gregorjohannson
martinpaljak#153: Add initial functionality of including DM token cal…
22ac666 
…culation for INSTALL [for load]. Minor refactoring for more clarity.
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Feb 7, 2019
@gregorjohannson
martinpaljak#153: Add initial functionality of including DM token cal…
f99b873 
…culation for INSTALL [for load]. Minor refactoring for more clarity.
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Feb 13, 2019
@gregorjohannson
martinpaljak#153: Extract DM token addition to separate universal han…
21b1865 
…dler, use in all commands that require DM token. Replace private key loading implementation.
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Feb 13, 2019
@gregorjohannson
martinpaljak#153: Simple tests for DelegatedManagementHandler
d3ba331
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Feb 19, 2019
@gregorjohannson
martinpaljak#153: Refactor DAP properties assignment to separate clas…
d480e4e 
…s. Handle DAP in --install. Extract duplicated CAP loading to separate method. Declare DM token in GlobalPlatform and load it in transitDM. Refactoring.
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Feb 19, 2019
@gregorjohannson
martinpaljak#153: Sign APDU data for token with crypto.Cipher (RSA/EC…
3338d5c 
…B/PKCS1Padding) instead of security.Signature
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Feb 21, 2019
@gregorjohannson
martinpaljak#153: Fix introduced LGTM alert. Remove DAPProperties set…
a0a18a8 
…ters. Cipher back to Signature. Swap -token to -token-key.
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Feb 21, 2019
@gregorjohannson
martinpaljak#153: Fix another LGTM error
dfce1b7
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Mar 21, 2019
@gregorjohannson
martinpaljak#153: Fix false assumption that INS_DELETE does not requi…
11e6b9d 
…re zero-length of DM token to be transmitted
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Mar 27, 2019
@gregorjohannson
martinpaljak#153: Include required LFDB hash calculation for load if…
53b9781 
… token-key exists (as per specification). Respect SHA256 option for load if DAP not required.
gregorjohannson added a commit to gregorjohannson/GlobalPlatformPro that referenced this issue Mar 27, 2019
@gregorjohannson
martinpaljak#153: Add tag '9E' before Delete Token
8f2788e
martinpaljak pushed a commit that referenced this issue May 3, 2019
@gregorjohannson @martinpaljak
#153 Delegated management token support (#155)
c8dad37 
* #153: Add initial functionality of including DM token calculation for INSTALL [for load]. Minor refactoring for more clarity.
* #153: Extract DM token addition to separate universal handler, use in all commands that require DM token. Replace private key loading implementation.
* #153: Simple tests for DelegatedManagementHandler
* #153: Refactor DAP properties assignment to separate class. Handle DAP in --install. Extract duplicated CAP loading to separate method. Declare DM token in GlobalPlatform and load it in transitDM. Refactoring.
* #153: Sign APDU data for token with crypto.Cipher (RSA/ECB/PKCS1Padding) instead of security.Signature
* #153: Fix introduced LGTM alert. Remove DAPProperties setters. Cipher back to Signature. Swap -token to -token-key.
* #153: Fix another LGTM error
* #153: Fix false assumption that INS_DELETE does not require zero-length of DM token to be transmitted
*  #153: Include required LFDB hash calculation for load if token-key exists (as per specification). Respect SHA256 option for load if DAP not required.
*  #153: Add tag '9E' before Delete Token
@martinpaljak martinpaljak mentioned this issue May 16, 2019
Merged
martinpaljak added a commit that referenced this issue May 16, 2019
@martinpaljak
Major refactoring
0014bda 
- Sanitize key handling, splitting card keys and session keys
- Merge registry elements into single class
- Rename things to be more uniform
- Remove a lot of rot
- Release a snapshot with updated dependencies

closes #118 #165 #153 #9
@martinpaljak
Copy link
Owner Author

This has been finalized in #215

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@martinpaljak