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

API 1_SUCCESS_TEST jcardsim #7

Open
wants to merge 47 commits into
base: master
Choose a base branch
from
Open

API 1_SUCCESS_TEST jcardsim #7

wants to merge 47 commits into from

Conversation

typelogic
Copy link
Contributor

No description provided.

@typelogic typelogic requested a review from jeremi January 3, 2020 10:41
@typelogic
Copy link
Contributor Author

Design goals in mind, as practically as possible:

  • A unified API abstraction flow (jcardsim, desktop-pcsc, android-nfc, other types of communication)
  • Support multiple non-default keys
  • Use most minimal java language features

dexter added 14 commits January 15, 2020 22:47
Change static to singleton instance; transfer secure channel from app…
7e12484 
…let to OffCard
Changes for a unified design, common code between physical and simulator
00f46f6
Delete Physical.java because OffCard.java is physical, one-less state…
c13d157 
… to worry about.
Consistent variable naming
2821e52
Stronger type check with enum instead of System.getProperty()
8cf95a2
Add keyset management plumbing.
dad898d
Completely start new jcardsim per test case
fb22fcf
Better placement of applet's simulated secureChannel from OffCard.jav…
7c843bd 
…a to DummyIssuerSecurityDomain.java
Disable physical test in circleci
a46ebe9
Rename method names to match mirror classes
260d587
Add readme content; changes intended to increase code symmetry
b170396
Add logic in gradle for smoother testing
db0f775
Set CLA to 0x00 as OffCard can set new CLA based on scp02
207a183
Remove unwanted try/catch blocks
e9af9d7
dexter added 19 commits January 22, 2020 19:17
Fix typo in a CommandAPDU
a6746d1
Gradually pulling-out inner assertions to be able to express these in…
8d056fd 
… the test cases
Remove filter of gradle test
b4e927a
Add protection check in circleci no PCSC
34857f3
Add card-sign-applet repo
5dbfd6b
Implement scp02 unwrap; Add more physical test cases
b25dffc
Fix typo verifierTemplateData
d3e59b6
Normalize return array to be []; Add TODO comments; Rename to Dump.java
dc5c905
Updating track of card-sign-applet submodule
36b1052
Fix install params on physical card testing
205197d
Add physical card signing test case to check JCSystem.CLEAR_ON_RESET …
6fbe00d 
…behavior
Add signTransactionTest Ethereum signing test case for SignApplet::si…
21078ab 
…gn APDU
Use GPSystem.SecureChannel security level bit flags for readability; …
21ac358 
…add latest GlobalPlatform reference PDF
Pin these constants in SCP02 for readability
6ff2879
Readability: Unscope from SCP02 to express clearly that value is from…
a4e4cfe 
… org.globalplatform.SecureChannel
Remove hardcoded pub/priv key
5dd3250
Add Data Element TLV encoding
51b240f
TLV testing on physical card using dev applet
0a9ebe7
Add TLV construct
428da6e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeremi jeremi Awaiting requested review from jeremi

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@typelogic