Skip to content

Cap transaction history count and update Google auth dependency#27

Open
afurm wants to merge 1 commit intosolana-developers:mainfrom
afurm:af/cap-transactions-last-count
Open

Cap transaction history count and update Google auth dependency#27
afurm wants to merge 1 commit intosolana-developers:mainfrom
afurm:af/cap-transactions-last-count

Conversation

@afurm
Copy link
Copy Markdown

@afurm afurm commented Apr 27, 2026

Summary

  • Cap GET /api/transactions/last?count= at 100 results.
  • Add route-level and e2e regression coverage for oversized count values.
  • Document the new count maximum in docs/API.md.
  • Update google-auth-library to 10.6.2 and refresh package-lock.json.

Why

/transactions/last accepted any positive integer for count, so an authorized caller could request an unbounded number of transaction rows. Capping the value keeps the endpoint predictable and limits unnecessary database and response work.

The dependency update removes the production gaxios -> uuid advisory reported by npm audit --omit=dev.

Validation

  • npm test
  • npm audit --omit=dev

Note: npm run test:e2e could not run locally because Testcontainers could not find a working container runtime in this environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant