fix: do not try to use OpenSSL when building for Electron

Author: addaleax

Diff for: addon/openssl-crypto.cc

@@ -5,6 +5,12 @@
 // This file provides native crypto hooks for OpenSSL 3 (the default since Node.js 18),
 // allowing us to skip expensive round-trips between JS and C++.
 
+#include "mongocrypt.h"
+
+// Electron does not expose OpenSSL, so we cannot use OpenSSL
+// functions directly if we're building against Electron:
+// https://github.com/electron/electron/issues/13176
+#ifndef MONGOCRYPT_AVOID_OPENSSL_CRYPTO
 #include <openssl/crypto.h>
 #include <openssl/err.h>
 #include <openssl/evp.h>
@@ -13,8 +19,6 @@
 
 #include <stdexcept>
 
-#include "mongocrypt.h"
-
 #ifdef _WIN32
 #include <windows.h>
 #else
@@ -441,3 +445,13 @@ std::unique_ptr<CryptoHooks> createOpenSSLCryptoHooks() {
 
 }  // namespace opensslcrypto
 }  // namespace node_mongocrypt
+
+#else // MONGOCRYPT_AVOID_OPENSSL_CRYPTO
+namespace node_mongocrypt {
+namespace opensslcrypto {
+std::unique_ptr<CryptoHooks> createOpenSSLCryptoHooks() {
+    return {};
+}
+}
+}
+#endif // MONGOCRYPT_AVOID_OPENSSL_CRYPTO

Diff for: binding.gyp

@@ -8,6 +8,8 @@
     'variables': {
       'ARCH': '<(host_arch)',
       'libmongocrypt_link_type%': 'static',
+      'mongocrypt_avoid_openssl_crypto%': 'false',
+      'built_with_electron%': 0
     },
     'sources': [
       'addon/mongocrypt.cc',
@@ -25,6 +27,7 @@
       'VCCLCompilerTool': { 'ExceptionHandling': 1 },
     },
     'conditions': [
+      ['mongocrypt_avoid_openssl_crypto=="true" or built_with_electron==1', { 'defines': ['MONGOCRYPT_AVOID_OPENSSL_CRYPTO'] }],
       ['OS=="mac"', { 'cflags+': ['-fvisibility=hidden'] }],
       ['_type!="static_library" and ARCH=="arm64"', {
           'xcode_settings': {