Merge pull request #41 from ar-io/fix-remove-primary-names

fix(primary names): update Remove-Primary-Names message to use comma delimited list
ar-io · Nov 25, 2024 · 28c6c26 · 28c6c26
2 parents 138f712 + d6f8bad
commit 28c6c26
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/common/main.lua b/src/common/main.lua
@@ -309,7 +309,7 @@ function ant.init()
 		ao.send({
 			Target = ioProcess,
 			Action = "Remove-Primary-Names",
-			Names = json.encode(names),
+			Names = msg.Tags.Names,
 		})
 	end)
 end

diff --git a/test/primary_names.test.mjs b/test/primary_names.test.mjs
@@ -85,7 +85,7 @@ describe('Primary Names', async () => {
 
     assert.strictEqual(removePrimaryNamesMsg.Target, ''.padEnd(43, '2'));
     assert.strictEqual(actionTag.value, 'Remove-Primary-Names');
-    assert.strictEqual(namesTag.value, JSON.stringify(names));
+    assert.strictEqual(namesTag.value, names.join(','));
   });
   it('should not send remove names request if caller not owner', async () => {
     const names = ['foo', 'bar', 'baz'];