Skip to content

Add Enclave Simulator for Internal Testing - #427

Merged
karinazhou merged 2 commits into
dotnet:masterfrom
karinazhou:optional_enclave_simulator
Feb 14, 2020
Merged

Add Enclave Simulator for Internal Testing#427
karinazhou merged 2 commits into
dotnet:masterfrom
karinazhou:optional_enclave_simulator

Conversation

@karinazhou

Copy link
Copy Markdown
Member
  • Added support for enclave simulator bypassing the actual attestation.
  • Added BuildSimulator property to enable the simulator support.

How to use the enclave simulator:

Given Attestation Protocol = SIM and Enclave Attestation Url= dummyURL in the connection string and run the AE enclave-enabled tests.

NOTE: This change is for internal testing only. The simulator support should be disabled in the official nuget package releases.

@David-EngelDavid-Engel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@karinazhou
karinazhou merged commit 66edd58 into dotnet:masterFeb 14, 2020
cheenamalhotra pushed a commit to cheenamalhotra/SqlClient that referenced this pull request Mar 3, 2020
- Added support for enclave simulator bypassing the actual attestation.
- Added BuildSimulator property to enable the simulator support.
How to use the enclave simulator:
Given Attestation Protocol = SIM and Enclave Attestation Url= SomeDummyURL in the connection string and run the AE enclave-enabled tests.
NOTE: This change is for internal testing only. The simulator support should be disabled in the official nuget package releases.
@cheenamalhotracheenamalhotra added this to the 2.0.0-preview2 milestone Mar 23, 2020
@cheenamalhotracheenamalhotra added the Area\Tests Issues that are targeted to tests or test projects label Mar 23, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area\TestsIssues that are targeted to tests or test projects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@karinazhou@cheenamalhotra@David-Engel