Description

getEventExpiration() in src/utils/event.ts used Math.log10(expirationTime) < 10 as a bounds check on the NIP-40 expiration tag. This has two bugs:

• Math.log10(0) === -Infinity, which is < 10, so an expiration of 0 (Unix epoch) was accepted as valid.
• Any safe-integer timestamp past year 2287 (> 9,999,999,999) failed the < 10 check and was silently discarded, treated as "no expiration."

Replaced the check with expirationTime > 0, which accepts any positive safe integer and rejects non-positive values, with no arbitrary digit-count ceiling.

Related Issue

Closes #707

Motivation and Context

An expiration of 0 should never be treated as a valid future expiration, and a relay shouldn't silently ignore legitimate (if far-future) expiration timestamps just because they cross an arbitrary digit-count boundary.

How Has This Been Tested?

• Added two unit tests in test/unit/utils/event.spec.ts under the existing NIP-40 > getEventExpiration suite:
  • expiration: '0' → undefined
  • expiration: '10000000000' (year 2287+) → returned as-is
• Ran pnpm run test:unit (1395 passing) and pnpm run test:cli (73 passing)
• Ran pnpm run lint, pnpm run check:deps, pnpm run build,
  pnpm run verify:cli:build, pnpm run cover:unit — all pass.

Screenshots (if appropriate):

N/A — pure logic fix, no UI/API surface change.

Types of changes

• Non-functional change (docs, style, minor refactor)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my code changes.
• I added a changeset, or this is docs-only and I added an empty changeset.
• All new and existing tests passed.