-
-
Notifications
You must be signed in to change notification settings - Fork 104
Open
Labels
good first issueGood for newcomersGood for newcomers
Description
Summary
The lendingPoolConfigureBankInterestOnly instruction currently logs the parameter changes but not the bank pubkey being configured. Adding the bank address to the logs would improve transaction verification in multisig workflows.
Current Behavior
When executing lendingPoolConfigureBankInterestOnly, the program logs show:
Program log: Setting self.optimal_utilization_rate to 0.8
Program log: Setting self.plateau_interest_rate to 0.06
Program log: Bank configured!
Desired Behavior
The logs should include the bank pubkey being configured:
Program log: Configuring bank: 2s37akK2eyBbp8DZgCm7RtsaEz8eJP3Nxd4urLHQv7yB
Program log: Setting self.optimal_utilization_rate to 0.8
Program log: Setting self.plateau_interest_rate to 0.06
Program log: Bank configured!
Why This Matters
Having the bank pubkey in logs allows multisig reviewers to verify they're configuring the correct bank and look up its token, group, and current configuration.
Safety
This should be safe to add since the instruction currently consumes ~5,132 CUs and memory usage is presumably very low, but worth verifying there's no edge case impact.
Affected Instructions
lending_pool_configure_bank_interest_only
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomers