draft: adding handler to return all assets of an account #5290
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
A half attempt at an endpoint to support
#5250
Mostly because I was curious about what it would take and to serve as a subject for discussion
If we were to pursue this approach we'd probably want to at least
/0
in the path so that the endpoint is just/v2/accounts/{address}/assets
If performance of this endpoint is a concern, we could add it to one of the flagged off set of handlers
Test Plan
copied the bin into my sandbox and ran a curl
curl localhost:4001/v2/accounts/AZRSDXCKQIUKIEEFF7FMF43ZNTFWCYONKAI3SXX5GRVPURLMTXI2OVSKUY/assets/0 -H "X-Algo-API-Token: aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"
returns