cerc-io/laconicd
18
0
Fork 2
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases 10 Wiki Activity

Fixes for handling authority expiry queue #40

Merged
ashwin merged 4 commits from pm-fix-authority-reservation into main 2024-07-22 05:40:46 +00:00
Conversation 0 Commits 4 Files Changed 2 +8 -2
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
x/registry/keeper/naming_keeper.go
View File

@ -519,7 +519,8 @@ func (k Keeper) ProcessAuthorityExpiryQueue(ctx sdk.Context) error {
k.Logger(ctx).Info(fmt.Sprintf("Marking authority expired as no bond present: %s", name)) k.Logger(ctx).Info(fmt.Sprintf("Marking authority expired as no bond present: %s", name))
return nil // Continue with the loop
continue
} }
// Try to renew the authority by taking rent. // Try to renew the authority by taking rent.
@ -585,7 +586,7 @@ func (k Keeper) deleteAuthorityExpiryQueue(ctx sdk.Context, name string, authori
} }
} }
if len(existingNamesList.Value) == 0 { if len(newNamesSlice) == 0 {
return k.AuthorityExpiryQueue.Remove(ctx, expiryTime) return k.AuthorityExpiryQueue.Remove(ctx, expiryTime)
} else { } else {
existingNamesList.Value = newNamesSlice existingNamesList.Value = newNamesSlice

5
x/registry/keeper/record_keeper.go
View File

@ -111,6 +111,11 @@ func (rk RecordKeeper) OnAuctionWinnerSelected(ctx sdk.Context, auctionId string
// Mark as expired. // Mark as expired.
authority.Status = registrytypes.AuthorityExpired authority.Status = registrytypes.AuthorityExpired
logger(ctx).Info(fmt.Sprintf("No winner, marking authority as expired: %s", name)) logger(ctx).Info(fmt.Sprintf("No winner, marking authority as expired: %s", name))
logger(ctx).Info(fmt.Sprintf("Deleting the expiry queue entry: %s", name))
if err = rk.k.deleteAuthorityExpiryQueue(ctx, name, authority); err != nil {
logger(ctx).Error("Unable to delete expiry queue entry", err)
}
} }
// Forget about this auction now, we no longer need it. // Forget about this auction now, we no longer need it.