Merge pull request #246 from ar-io/reserved-name-fix

fix(reserved): mark name as reserved if it does not have an endtimestamp
ar-io · May 22, 2024 · bbef928 · bbef928
2 parents ff5eee4 + 6f9e9f7
commit bbef928
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/actions/write/buyRecord.test.ts b/src/actions/write/buyRecord.test.ts
@@ -36,7 +36,7 @@ describe('buyRecord', () => {
   );
 
   it.each([['a-expired-reserved-name', 'not-short-name', 'reserved']])(
-    'should be able to buy a record of a reserved name when the caller is the target of the reserved name',
+    'should be able to buy a record of a reserved name when the caller is not the target of the reserved name but it is expired',
     async (reservedName: string) => {
       const initialState = {
         ...getBaselineState(),

diff --git a/src/records.ts b/src/records.ts
@@ -119,7 +119,9 @@ export function isActiveReservedName({
 
   const isCallerTarget = caller !== undefined && target === caller;
   const isActiveReservation =
-    endTimestamp && endTimestamp > currentBlockTimestamp.valueOf();
+    endTimestamp === undefined ||
+    (endTimestamp !== undefined &&
+      endTimestamp > currentBlockTimestamp.valueOf());
 
   // if the caller is not the target, and it's still active - the name is considered reserved
   if (!isCallerTarget && isActiveReservation) {