remove pem restriction for now

apple-platform-rs orders their pem certs with key then cert...
nodeselector · Aug 2, 2024 · adb4d93 · adb4d93
1 parent d8c8f51
commit adb4d93
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/src/certificate.ts b/src/certificate.ts
@@ -19,14 +19,14 @@ export async function prepareKeychainWithDeveloperCertificate(
   keychain: Keychain
 ): Promise<void> {
   const decodedCert = Buffer.from(secretValue, 'base64').toString('utf-8')
-  // TODO this won't work for all certificate encodings (e.g. DER)
-  try {
-    pki.certificateFromPem(decodedCert)
-  } catch (e) {
-    throw new Error(
-      `Failed to parse certificate (PEM support only right now): ${e}`
-    )
-  }
+  // // TODO this won't work for all certificate encodings (e.g. DER)
+  // try {
+  //   pki.certificateFromPem(decodedCert)
+  // } catch (e) {
+  //   throw new Error(
+  //     `Failed to parse certificate (PEM support only right now): ${e}`
+  //   )
+  // }
 
   await keychain.createKeychain()
   await keychain.addToSearchList()