Error at SET if space in key or value

[tripower]

Hello

we get the following ERROR if a space is in the key or value of the set method
ProcessUncleanExitError: ADD command exited with code 1:
FEHLER: Ungültige Syntax.
Geben Sie "REG ADD /?" ein, um die Syntax anzuzeigen.

method call:
key.set("URL Protocol", WinReg.REG_SZ, '', (err)=> {
});

or

key2.set(WinReg.DEFAULT_VALUE, WinReg.REG_SZ, '"ExeFileFolder\ExeFileName" "--deeplink=%1"', (err)=> {
});

Info:
it works on CMD
`
C:>reg add HKCU\SOFTWARE\Classes\testKB /v "URL Protocol" /t REG_SZ /d "" /f
Der Vorgang wurde erfolgreich beendet.

C:>reg query HKCU\SOFTWARE\Classes\testKB /v "URL Protocol"
HKEY_CURRENT_USER\SOFTWARE\Classes\testKB
URL Protocol REG_SZ
`

update 1:
key.set("\"URL Protocol\"",..
results in exteme cpu usage and forever running thread

update 2:
version 1.2.4 works!!! (with pure NodeJS and in my electron v18 app)
version 1.2.5 also NOT ok if I took the set method from 1.2.4 (very strange)
so I do not close the issue because 1.2.5 should also work

[andymartinwork]

This happened to me too. 1.2.5 stopped working with this error:

ERROR: Invalid syntax. Type "REG ADD /?" for usage.

[chunkbanned]

I also have the same issue on 1.2.5, and when downgrading back to 1.2.4 it works perfectly fine with no issues.
This message showed ERROR: Invalid syntax. Type "REG ADD /?" for usage.

[jazpearson]

Wow, me too. This is particularly nasty.

[neriyaco]

It looks like this package is not maintained anymore, I used this package's code and made a Promise base library that also fixes this issue.
You can check it out here

If you prefer to use this package you can use patch-package with this patch to fix this issue:

diff --git a/node_modules/winreg/lib/registry.js b/node_modules/winreg/lib/registry.js
index 6319ae3..1cf5546 100644
--- a/node_modules/winreg/lib/registry.js
+++ b/node_modules/winreg/lib/registry.js
@@ -1,3 +1,12 @@
+/*
+ * This file is licensed under the BSD 2-Clause License.
+ *
+ * BSD 2-Clause License:
+ * Copyright (c) 2016, Paul Bottin All rights reserved.
+ *
+ * See the LICENSE file in the project root for full license information.
+ */
+
 /************************************************************************************************************
  * registry.js - contains a wrapper for the REG command under Windows, which provides access to the registry
  *
@@ -613,7 +622,7 @@ Registry.prototype.get = function get (name, cb) {
   if (name == '')
     args.push('/ve');
   else
-    args = args.concat(['/v', name]);
+    args = args.concat(['/v', '"' + name + '"']);
 
   pushArch(args, this.arch);
 
@@ -704,9 +713,13 @@ Registry.prototype.set = function set (name, type, value, cb) {
   if (name == '')
     args.push('/ve');
   else
-    args = args.concat(['/v', name]);
+    args = args.concat(['/v', '"' + name + '"']);
 
-  args = args.concat(['/t', type, '/d', value, '/f']);
+  args = args.concat(['/t', type]);
+  if (value !== '') {
+    args = args.concat(['/d', '"' + value + '"']);
+  }
+  args.push('/f');
 
   pushArch(args, this.arch);
 
@@ -758,7 +771,7 @@ Registry.prototype.remove = function remove (name, cb) {
   if (typeof cb !== 'function')
     throw new TypeError('must specify a callback');
 
-  var args = name ? ['DELETE', this.path, '/f', '/v', name] : ['DELETE', this.path, '/f', '/ve'];
+  var args = name ? ['DELETE', this.path, '/f', '/v', '"' + name + '"'] : ['DELETE', this.path, '/f', '/ve'];
 
   pushArch(args, this.arch);

[panther7]

This package is dead, because fixed PR is available over 2 years...

• Fixed create key spaces #61
• Can't create key, if name or value have spaces #60