search

lnbits / lnurlp

MIT License
9 stars 8 forks source link

Install fails on lnbits docker #29

Closed ThomasFarstrike closed 1 year ago

ThomasFarstrike commented 1 year ago

Using the latest docker image (docker pull lnbitsdocker/lnbits-legend:0.11.0), I am able to install most extensions, including lnurlw, but lnurlp fails:

docker lnbits logs says:

2023-10-14 05:52:31.39 | INFO | 172.17.0.1:50356 - "GET /extensions?usr=XXXXX HTTP/1.1" 200
2023-10-14 05:52:37.31 | INFO | 172.17.0.1:49686 - "GET /api/v1/extension/lnurlp/releases?usr=XXXXX HTTP/1.1" 200
2023-10-14 05:52:42.41 | INFO | lnbits.extension_manager:download_archive:377 | Downloading extension lnurlp (0.3.3).
2023-10-14 05:52:43.25 | INFO | lnbits.extension_manager:extract_archive:402 | Extracting extension lnurlp (0.3.3).
2023-10-14 05:52:43.28 | SUCCESS | lnbits.extension_manager:extract_archive:439 | Extension LNURLp (0.3.3) installed.
2023-10-14 05:52:43.30 | ERROR | lnbits.core.helpers:migrate_extension_database:22 | No module named 'environs'
2023-10-14 05:52:43.30 | WARNING | lnbits.core.views.api:api_install_extension:814 | Please make sure that the extension `lnurlp` has a migrations file.
2023-10-14 05:52:43.31 | ERROR | lnbits.app:http_exception_handler:501 | HTTPException 500: Failed to install extension lnurlp (0.3.3).
2023-10-14 05:52:43.31 | INFO | 172.17.0.1:49686 - "POST /api/v1/extension?usr=XXXXX HTTP/1.1" 500

Even though it says "Extension [...] installed.", the extension does not show up in the "installed" tab.

Here's a successful install of lnurlw for comparison:

2023-10-14 06:14:00.30 | INFO | 172.17.0.1:43794 - "GET /api/v1/extension/withdraw/releases?usr=XXXXX HTTP/1.1" 200
2023-10-14 06:14:04.10 | INFO | lnbits.extension_manager:download_archive:377 | Downloading extension withdraw (0.1.4).
2023-10-14 06:14:04.82 | INFO | lnbits.extension_manager:extract_archive:402 | Extracting extension withdraw (0.1.4).
2023-10-14 06:14:04.83 | SUCCESS | lnbits.extension_manager:extract_archive:439 | Extension LNURLw (0.1.4) installed.
2023-10-14 06:14:04.88 | DEBUG | lnbits.core.helpers:run_migration:39 | running migration withdraw.1
running migration withdraw.1
2023-10-14 06:14:04.90 | DEBUG | lnbits.core.helpers:run_migration:39 | running migration withdraw.2
running migration withdraw.2
2023-10-14 06:14:04.94 | DEBUG | lnbits.core.helpers:run_migration:39 | running migration withdraw.3
running migration withdraw.3
2023-10-14 06:14:04.96 | DEBUG | lnbits.core.helpers:run_migration:39 | running migration withdraw.4
running migration withdraw.4
2023-10-14 06:14:04.99 | DEBUG | lnbits.core.helpers:run_migration:39 | running migration withdraw.5
running migration withdraw.5
2023-10-14 06:14:05.02 | DEBUG | lnbits.core.helpers:run_migration:39 | running migration withdraw.6
running migration withdraw.6
2023-10-14 06:14:05.09 | INFO | 127.0.0.1:46720 - "DELETE /withdraw/api/v1?usr=XXXXX HTTP/1.1" 404
2023-10-14 06:14:05.12 | INFO | 172.17.0.1:43794 - "POST /api/v1/extension?usr=XXXXX HTTP/1.1" 200
2023-10-14 06:14:05.40 | INFO | 172.17.0.1:43794 - "GET /extensions?usr=XXXXX&activate=withdraw HTTP/1.1" 200

This one does show up in the "installed" tab.

ThomasFarstrike commented 1 year ago

Aaah it's because this fix was made after the 0.11.0 release: https://github.com/lnbits/lnbits/pull/2007