From aeb6eac840a5c7be9ce5d688b9989a0221db1247 Mon Sep 17 00:00:00 2001 From: Bryan Stitt Date: Tue, 22 Nov 2022 05:42:02 +0000 Subject: [PATCH] comments/todos --- TODO.md | 7 +++++-- web3_proxy/src/bin/web3_proxy_cli/main.rs | 7 ++++--- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/TODO.md b/TODO.md index 0a09e0fe..e2d2ed87 100644 --- a/TODO.md +++ b/TODO.md @@ -242,15 +242,18 @@ These are roughly in order of completition - [x] cli command to change user_tier by key - [x] cache the status page for a second - [x] request accounting for websockets -- [ ] database merge scripts +- [x] database merge scripts - [ ] add block timestamp to the /status page - [ ] be sure to save the timestamp in a way that our request routing logic can make use of it +- [ ] period_datetime should always be :00. right now it depends on start time +- [ ] two servers running will confuse rpc_accounting! + - one option: we need the insert to be an upsert, but how do we merge historgrams? - [ ] change invite codes to set the user_tier - [ ] period_datetime should always be :00. right now it depends on start time - [ ] two servers running will confuse rpc_accounting! - it won't happen with users often because they should be sticky to one proxy, but unauthenticated users will definitely hit this - one option: we need the insert to be an upsert, but how do we merge historgrams? - +- [ ] ethspam on bsc and polygon gives 1/4 errors. fix whatever is causing this - [ ] actually block unauthenticated requests instead of emitting warning of "allowing without auth during development!" diff --git a/web3_proxy/src/bin/web3_proxy_cli/main.rs b/web3_proxy/src/bin/web3_proxy_cli/main.rs index 380eadc8..9fb4db2e 100644 --- a/web3_proxy/src/bin/web3_proxy_cli/main.rs +++ b/web3_proxy/src/bin/web3_proxy_cli/main.rs @@ -40,9 +40,10 @@ enum SubCommand { CreateUser(create_user::CreateUserSubCommand), DropMigrationLock(drop_migration_lock::DropMigrationLockSubCommand), UserExport(user_export::UserExportSubCommand), - UserImport(user_import::UserImportSubCommand), // TODO: sub command to downgrade migrations? - // TODO: sub command to add new api keys to an existing user? - // TODO: sub command to change a user's tier + UserImport(user_import::UserImportSubCommand), + // TODO: sub command to downgrade migrations? sea-orm has this but doing downgrades here would be easier+safer + // TODO: sub command to add new api keys to an existing user? + // TODO: sub command to change a user's tier } #[tokio::main]