Skip to content

txHandler: use txn.Access data when calculating app usage rates#6531

Merged
algorandskiy merged 4 commits intoalgorand:masterfrom
algorandskiy:pavel/txhandler-arl-txnrefs
Jan 28, 2026
Merged

txHandler: use txn.Access data when calculating app usage rates#6531
algorandskiy merged 4 commits intoalgorand:masterfrom
algorandskiy:pavel/txhandler-arl-txnrefs

Conversation

@algorandskiy
Copy link
Copy Markdown
Contributor

Summary

While adding txn.Access in #6286 we've forgot to update txHandler and its ARL app counting logic. This PR fixes that.

Test Plan

Added a unit test

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.68%. Comparing base (13169aa) to head (e544b78).
⚠️ Report is 2 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6531      +/-   ##
==========================================
- Coverage   47.87%   47.68%   -0.20%     
==========================================
  Files         662      655       -7     
  Lines       87955    87871      -84     
==========================================
- Hits        42110    41897     -213     
- Misses      43083    43198     +115     
- Partials     2762     2776      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

gmalouf
gmalouf previously approved these changes Jan 26, 2026
Comment thread data/appRateLimiter.go Outdated
Comment thread data/appRateLimiter.go Outdated
Comment thread data/appRateLimiter.go Outdated
Comment thread data/appRateLimiter.go Outdated
@algorandskiy algorandskiy requested a review from jannotti January 27, 2026 18:44
Comment thread data/appRateLimiter_test.go Outdated
@gmalouf gmalouf requested a review from jannotti January 27, 2026 19:20
@algorandskiy algorandskiy merged commit 2626446 into algorand:master Jan 28, 2026
39 checks passed
@algorandskiy algorandskiy deleted the pavel/txhandler-arl-txnrefs branch March 16, 2026 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants