Skip to content

Fix scan findings: price_aggregates table, env vars, zero-div checks - #10

Merged
buzzkillb merged 15 commits into
mainfrom
dev
Mar 17, 2026
Merged

Fix scan findings: price_aggregates table, env vars, zero-div checks#10
buzzkillb merged 15 commits into
mainfrom
dev

Fix duplicate test function in utils.rs

255240b
Select commit
Loading
Failed to load commit list.
GitzillaBot / gitzilla succeeded Mar 17, 2026 in 2m 3s

Gitzilla review complete

No actionable issues found.

This pull request adds the missing price_aggregates table to the database schema initialization in database.rs. It modifies the Shanghai price service to configure its update interval via the UPDATE_INTERVAL_SECONDS environment variable instead of using a hardcoded value. Additionally, it inserts a zero-check before division in the bot's price calculation logic to prevent potential runtime errors when the western price is zero. Lastly, it removes a duplicate comment in bot.rs for improved code clarity.

Details

No actionable issues found in this review pass.