From ab19685a2a20a18a50c1b0711a5e03f867cda7a3 Mon Sep 17 00:00:00 2001 From: padreug Date: Wed, 15 Oct 2025 00:56:14 +0200 Subject: [PATCH 1/4] feat: implement automatic crediting of new accounts with 1 million satoshis MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Modified the user account creation process to automatically credit new accounts with 1,000,000 satoshis upon creation. - Updated the `create_user_account_no_ckeck` function to include error handling and logging for the credit operation. - Enhanced tests to verify the balance for newly registered users, ensuring the correct credit amount is applied. - Documented affected account creation paths and testing instructions in the new `AUTO_CREDIT_CHANGES.md` file. fix: pass db connection to update_wallet_balance to prevent SQLite deadlock The auto-credit feature was causing a deadlock on SQLite because update_wallet_balance was opening a new database connection while already inside a transaction. By passing conn=conn, we reuse the existing connection and avoid the deadlock. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- AUTO_CREDIT_CHANGES.md | 88 +++++++++++++++++++++++++++++++++++ lnbits/core/services/users.py | 9 ++++ tests/api/test_auth.py | 8 ++++ 3 files changed, 105 insertions(+) create mode 100644 AUTO_CREDIT_CHANGES.md diff --git a/AUTO_CREDIT_CHANGES.md b/AUTO_CREDIT_CHANGES.md new file mode 100644 index 00000000..408f6987 --- /dev/null +++ b/AUTO_CREDIT_CHANGES.md @@ -0,0 +1,88 @@ +# LNBits Auto-Credit Changes + +## Overview +Modified LNBits server to automatically credit new accounts with 1 million satoshis (1,000,000 sats) when they are created. + +## Changes Made + +### 1. Modified `lnbits/core/services/users.py` + +**Added imports:** +- `get_wallet` from `..crud` +- `update_wallet_balance` from `.payments` + +**Modified `create_user_account_no_ckeck` function:** +- Changed `create_wallet` call to capture the returned wallet object +- Added automatic credit of 1,000,000 sats after wallet creation +- Added error handling and logging for the credit operation + +**Code changes:** +```python +# Before: +await create_wallet( + user_id=account.id, + wallet_name=wallet_name or settings.lnbits_default_wallet_name, +) + +# After: +wallet = await create_wallet( + user_id=account.id, + wallet_name=wallet_name or settings.lnbits_default_wallet_name, +) + +# Credit new account with 1 million satoshis +try: + await update_wallet_balance(wallet, 1_000_000) + logger.info(f"Credited new account {account.id} with 1,000,000 sats") +except Exception as e: + logger.error(f"Failed to credit new account {account.id} with 1,000,000 sats: {e}") +``` + +### 2. Updated Tests in `tests/api/test_auth.py` + +**Modified test functions:** +- `test_register_ok`: Added balance verification for regular user registration +- `test_register_nostr_ok`: Added balance verification for Nostr authentication + +**Added assertions:** +```python +# Check that the wallet has 1 million satoshis +wallet = user.wallets[0] +assert wallet.balance == 1_000_000, f"Expected 1,000,000 sats balance, got {wallet.balance} sats" +``` + +## Affected Account Creation Paths + +The automatic credit will be applied to all new accounts created through: + +1. **Regular user registration** (`/api/v1/auth/register`) +2. **Nostr authentication** (`/api/v1/auth/nostr`) +3. **SSO login** (when new account is created) +4. **API account creation** (`/api/v1/account`) +5. **Admin user creation** (via admin interface) + +## Excluded Paths + +- **Superuser/Admin account creation** (`init_admin_settings`): This function creates the admin account directly and bypasses the user creation flow, so it won't receive the automatic credit. + +## Testing + +To test the changes: + +1. Install dependencies: `poetry install` +2. Run the modified tests: `poetry run pytest tests/api/test_auth.py::test_register_ok -v` +3. Run Nostr test: `poetry run pytest tests/api/test_auth.py::test_register_nostr_ok -v` + +## Logging + +The system will log: +- Success: `"Credited new account {account.id} with 1,000,000 sats"` +- Failure: `"Failed to credit new account {account.id} with 1,000,000 sats: {error}"` + +## Notes + +- The credit uses the existing `update_wallet_balance` function which creates an internal payment record +- The credit is applied after wallet creation but before user extensions are set up +- Error handling ensures that account creation continues even if the credit fails +- The credit amount is hardcoded to 1,000,000 sats (1MM sats) + diff --git a/lnbits/core/services/users.py b/lnbits/core/services/users.py index d21a8aff..16d6884c 100644 --- a/lnbits/core/services/users.py +++ b/lnbits/core/services/users.py @@ -26,6 +26,7 @@ from ..crud import ( get_super_settings, get_user_extensions, get_user_from_account, + get_wallet, update_account, update_super_user, update_user_extension, @@ -37,6 +38,7 @@ from ..models import ( UserExtra, ) from .settings import update_cached_settings +from .payments import update_wallet_balance async def create_user_account( @@ -80,6 +82,13 @@ async def create_user_account_no_ckeck( conn=conn, ) + # Credit new account with 1 million satoshis + try: + await update_wallet_balance(wallet, 1_000_000, conn=conn) + logger.info(f"Credited new account {account.id} with 1,000,000 sats") + except Exception as e: + logger.error(f"Failed to credit new account {account.id} with 1,000,000 sats: {e}") + user_extensions = (default_exts or []) + settings.lnbits_user_default_extensions for ext_id in user_extensions: try: diff --git a/tests/api/test_auth.py b/tests/api/test_auth.py index 57cacd03..8b1e7ba2 100644 --- a/tests/api/test_auth.py +++ b/tests/api/test_auth.py @@ -295,6 +295,10 @@ async def test_register_ok(http_client: AsyncClient): assert ( len(user.wallets) == 1 ), f"Expected 1 default wallet, not {len(user.wallets)}." + + # Check that the wallet has 1 million satoshis + wallet = user.wallets[0] + assert wallet.balance == 1_000_000, f"Expected 1,000,000 sats balance, got {wallet.balance} sats" @pytest.mark.anyio @@ -588,6 +592,10 @@ async def test_register_nostr_ok(http_client: AsyncClient, settings: Settings): assert ( len(user.wallets) == 1 ), f"Expected 1 default wallet, not {len(user.wallets)}." + + # Check that the wallet has 1 million satoshis + wallet = user.wallets[0] + assert wallet.balance == 1_000_000, f"Expected 1,000,000 sats balance, got {wallet.balance} sats" @pytest.mark.anyio From a21ce4fd92d582a5a8532c3d925efb7e0da2566a Mon Sep 17 00:00:00 2001 From: padreug Date: Sun, 11 Jan 2026 16:10:13 +0100 Subject: [PATCH 2/4] add satmachineadmi v0.0.4 --- extensions.json | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/extensions.json b/extensions.json index 5ef33282..09e3c5c3 100644 --- a/extensions.json +++ b/extensions.json @@ -4,16 +4,6 @@ "castle" ], "extensions": [ - { - "id": "satmachineadmin", - "repo": "https://git.atitlan.io/aiolabs/satmachineadmin", - "name": "Satoshi Machine Admin", - "version": "0.0.1", - "short_description": "Admin Dashboard for Satoshi Machine", - "icon": "https://git.atitlan.io/aiolabs/satmachineadmin/raw/branch/main/static/image/aio.png", - "archive": "https://git.atitlan.io/aiolabs/satmachineadmin/archive/v0.0.1.zip", - "hash": "e9b61de40e940f05aca460a0560cd3cc8b98f2f0d861e37cef13b046eaec0b7c" - }, { "id": "satmachineadmin", "repo": "https://git.atitlan.io/aiolabs/satmachineadmin", @@ -25,6 +15,16 @@ "archive": "https://git.atitlan.io/aiolabs/satmachineadmin/archive/v0.0.3.zip", "hash": "0cf520ee62037298320d8c0caa6f15d89858447d826af53472cb369023eda46d" }, + { + "id": "satmachineadmin", + "repo": "https://git.atitlan.io/aiolabs/satmachineadmin", + "name": "Satoshi Machine Admin", + "version": "0.0.4", + "short_description": "Admin Dashboard for Satoshi Machine", + "icon": "https://git.atitlan.io/aiolabs/satmachineadmin/raw/branch/main/static/image/aio.png", + "archive": "https://git.atitlan.io/aiolabs/satmachineadmin/archive/v0.0.4.zip", + "hash": "f5a6e33e6379984964cd0a4b25225770c4356c5c8a62c8e811b712775a180615" + }, { "id": "satmachineclient", "repo": "https://git.atitlan.io/aiolabs/satmachineclient", From 019e650bc246526da0969fae78655e62f0453012 Mon Sep 17 00:00:00 2001 From: Patrick Mulligan Date: Mon, 19 Jan 2026 06:31:49 -0500 Subject: [PATCH 3/4] CHORE: bump castle to v0.0.4 --- extensions.json | 30 ++++++++++-------------------- 1 file changed, 10 insertions(+), 20 deletions(-) diff --git a/extensions.json b/extensions.json index 09e3c5c3..ff5202c9 100644 --- a/extensions.json +++ b/extensions.json @@ -85,26 +85,6 @@ "archive": "https://git.atitlan.io/aiolabs/nostrrelay/archive/v0.0.2.zip", "hash": "6e353358314b740679e55e2ff98c7024a97c6df8bf27d1784e8cee448407bebd" }, - { - "id": "castle", - "repo": "https://git.atitlan.io/aiolabs/castle", - "name": "Castle", - "version": "0.0.1", - "short_description": "Castle Accounting", - "icon": "https://git.atitlan.io/aiolabs/castle/raw/branch/main/static/image/castle.png", - "archive": "https://git.atitlan.io/aiolabs/castle/archive/v0.0.1.zip", - "hash": "075a96a6f3151472302434fc50552346f7910f82b9ae1c6508d1bfd6db14607b" - }, - { - "id": "castle", - "repo": "https://git.atitlan.io/aiolabs/castle", - "name": "Castle", - "version": "0.0.2", - "short_description": "Castle Accounting", - "icon": "https://git.atitlan.io/aiolabs/castle/raw/branch/main/static/image/castle.png", - "archive": "https://git.atitlan.io/aiolabs/castle/archive/v0.0.2.zip", - "hash": "0653b7dbfa37da023c7f4fe976441fd6bef0d56fe6d08eb4ce6d4ec178632f4a" - }, { "id": "castle", "repo": "https://git.atitlan.io/aiolabs/castle", @@ -115,6 +95,16 @@ "archive": "https://git.atitlan.io/aiolabs/castle/archive/v0.0.3.zip", "hash": "b9640dbeec00274d8382b6d9e3b4250f356f002c047c8e380ff9acecb2058f39" }, + { + "id": "castle", + "repo": "https://git.atitlan.io/aiolabs/castle", + "name": "Castle", + "version": "0.0.4", + "short_description": "Castle Accounting", + "icon": "https://git.atitlan.io/aiolabs/castle/raw/branch/main/static/image/castle.png", + "archive": "https://git.atitlan.io/aiolabs/castle/archive/v0.0.4.zip", + "hash": "a5169440c47265b6bb8cf1cc2466a6b743bab00d16e18bd3a97956fc8ea3a069" + }, { "id": "lnurlp", "repo": "https://github.com/lnbits/lnurlp", From 12806995a59937fdf4f92bf4b35c689e96c74918 Mon Sep 17 00:00:00 2001 From: Patrick Mulligan Date: Mon, 19 Jan 2026 07:20:18 -0500 Subject: [PATCH 4/4] CHORE update castle repo v0.0.5 --- extensions.json | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/extensions.json b/extensions.json index ff5202c9..daac9700 100644 --- a/extensions.json +++ b/extensions.json @@ -85,16 +85,6 @@ "archive": "https://git.atitlan.io/aiolabs/nostrrelay/archive/v0.0.2.zip", "hash": "6e353358314b740679e55e2ff98c7024a97c6df8bf27d1784e8cee448407bebd" }, - { - "id": "castle", - "repo": "https://git.atitlan.io/aiolabs/castle", - "name": "Castle", - "version": "0.0.3", - "short_description": "Castle Accounting", - "icon": "https://git.atitlan.io/aiolabs/castle/raw/branch/main/static/image/castle.png", - "archive": "https://git.atitlan.io/aiolabs/castle/archive/v0.0.3.zip", - "hash": "b9640dbeec00274d8382b6d9e3b4250f356f002c047c8e380ff9acecb2058f39" - }, { "id": "castle", "repo": "https://git.atitlan.io/aiolabs/castle", @@ -105,6 +95,16 @@ "archive": "https://git.atitlan.io/aiolabs/castle/archive/v0.0.4.zip", "hash": "a5169440c47265b6bb8cf1cc2466a6b743bab00d16e18bd3a97956fc8ea3a069" }, + { + "id": "castle", + "repo": "https://git.atitlan.io/aiolabs/castle", + "name": "Castle", + "version": "0.0.5", + "short_description": "Castle Accounting", + "icon": "https://git.atitlan.io/aiolabs/castle/raw/branch/main/static/image/castle.png", + "archive": "https://git.atitlan.io/aiolabs/castle/archive/v0.0.5.zip", + "hash": "4c7c3683632cbdd6b1e810e5d1020ad6f03189414507c62d4dc9217813983c3c" + }, { "id": "lnurlp", "repo": "https://github.com/lnbits/lnurlp",