Fixing state provisioning without skip_ra flag (#2883)

* skipping getting quote data when using Epid attestation type

* refactoring skip logic on bitacross-worker

* small optimization, avoiding fetch the api when skip_ra is true

bitacross-worker/service/src/enclave/tls_ra.rs

@@ -83,7 +83,12 @@ pub fn enclave_request_state_provisioning<E: TlsRemoteAttestation + RemoteAttest
 
 	let stream = TcpStream::connect(addr).map_err(|e| Error::Other(Box::new(e)))?;
 
-	let quoting_enclave_target_info = if !skip_ra {
+	#[cfg(not(feature = "dcap"))]
+	let get_quote_data = false;
+	#[cfg(feature = "dcap")]
+	let get_quote_data = !skip_ra;
+
+	let quoting_enclave_target_info = if get_quote_data {
 		match enclave_api.qe_get_target_info() {
 			Ok(quote_size) => Some(quote_size),
 			Err(e) => return Err(e),
@@ -92,7 +97,7 @@ pub fn enclave_request_state_provisioning<E: TlsRemoteAttestation + RemoteAttest
 		None
 	};
 
-	let quote_size = if !skip_ra {
+	let quote_size = if get_quote_data {
 		match enclave_api.qe_get_quote_size() {
 			Ok(quote_size) => Some(quote_size),
 			Err(e) => return Err(e),

tee-worker/service/src/enclave/tls_ra.rs

@@ -81,7 +81,12 @@ pub fn enclave_request_state_provisioning<E: TlsRemoteAttestation + RemoteAttest
 
 	let stream = TcpStream::connect(addr).map_err(|e| Error::Other(Box::new(e)))?;
 
-	let quoting_enclave_target_info = if !skip_ra {
+	#[cfg(not(feature = "dcap"))]
+	let get_quote_data = false;
+	#[cfg(feature = "dcap")]
+	let get_quote_data = !skip_ra;
+
+	let quoting_enclave_target_info = if get_quote_data {
 		match enclave_api.qe_get_target_info() {
 			Ok(quote_size) => Some(quote_size),
 			Err(e) => return Err(e),
@@ -90,7 +95,7 @@ pub fn enclave_request_state_provisioning<E: TlsRemoteAttestation + RemoteAttest
 		None
 	};
 
-	let quote_size = if !skip_ra {
+	let quote_size = if get_quote_data {
 		match enclave_api.qe_get_quote_size() {
 			Ok(quote_size) => Some(quote_size),
 			Err(e) => return Err(e),