1
0
Commit Graph

13810 Commits

Author SHA1 Message Date
ThomasV
4dc7f4d331 release script: call build_docker_image.sh 2021-06-10 14:40:46 +00:00
ThomasV
8788b64504 Update release script with APK fresh clone instruction. Add android/fresh_clone to gitignore 2021-06-10 14:30:17 +00:00
ghost43
c43896fe6a Merge pull request #7263 from SomberNight/202104_android
reproducible build for Android apk
2021-06-10 13:19:41 +00:00
SomberNight
45cb8b5b02 android reproducibility: add Signal's "apkdiff.py" tool, and instructions 2021-06-09 18:34:15 +02:00
SomberNight
3c9346cce2 android build: attempt at reproducible builds 2021-06-09 18:34:09 +02:00
ThomasV
0f4f5e5fb7 Merge pull request #7341 from SomberNight/202106_issue7339
wallet: fix channels backups in .backup files
2021-06-09 17:38:50 +02:00
SomberNight
53d6eeb3f3 wallet: rm get_txout_address method 2021-06-08 16:45:30 +02:00
Nima
13e4424922 cmd: add sign with privkey and wallet (#7147)
* Remove check for sign with privkey in init_cmdline

* Add with_privkey and with_wallet variants of signtransaction command

* Add unit tests for signtransaction_with_privkey and signtransaction_with_wallet commands
2021-06-08 14:33:55 +00:00
SomberNight
34c2cb5220 wallet: fix channels backups in .backup files
This is a regression from 64a931f21e,
which introduced "onchain_channel_backups", and renamed
the old "channel_backups" key to "imported_channel_backups".

The `save_backup` method was not changed to use the new "imported_channel_backups" key,
so the channel backups are in the backup file but they are ignored.
2021-06-08 16:21:43 +02:00
SomberNight
7e6d65ec11 wallet: RBF batching to only consider RBF-opted-in txs, even if local
This is easier to understand; the special case is not worth it.

related https://github.com/spesmilo/electrum/issues/7298
2021-06-07 20:11:56 +02:00
Thorsten Hempel
fbf76af2c5 Update wallet backup access (#7300) 2021-06-07 16:46:25 +00:00
SomberNight
6c69c73a9f wallet: add address corruption tests for imported wallets too
related: https://github.com/spesmilo/electrum/issues/7338
2021-06-07 18:30:24 +02:00
SomberNight
a425ab0301 invoices/lnaddr: LNInvoice.from_bech32 now raises InvoiceError
rm LnAddressError

fixes https://github.com/spesmilo/electrum/issues/7321
related https://github.com/spesmilo/electrum/pull/7234
2021-06-07 14:46:30 +02:00
SomberNight
eb6b4580e8 fix tests: adapt to breaking change of ipaddress.ip_address in py3.9.5
fixes https://github.com/spesmilo/electrum/issues/7307

We can simply remove that test as we don't actually care whether the
leading zeroes are allowed.

see 60ce8f0be6
see https://bugs.python.org/issue36384
2021-06-05 08:01:38 +02:00
ghost43
e7c7a2a834 Merge pull request #7308 from bitromortac/2105-fix-cmd-reverse-swap
swaps: fix satoshi conversion bug
2021-06-05 05:35:46 +00:00
SomberNight
cd50472b18 kivy: (fix) forbid creating LN payreq if LN is disabled for wallet
fix #6346

see https://github.com/spesmilo/electrum/issues/6346#issuecomment-854655931
2021-06-04 14:11:19 +02:00
Thorsten Hempel
b990ff78a5 appimage: update libc6-dev package (#7299) 2021-06-04 08:36:23 +00:00
bitromortac
3c02f6b922 swaps: fix satoshi conversion bug 2021-05-19 07:27:54 +02:00
SomberNight
cad4e77853 fix prev 2021-05-15 06:57:59 +02:00
SomberNight
b2169b745e wallet: (fix) "batch_rbf" must not mutate LN funding txs
The "Batch RBF transactions" feature mutates existing "local" and "unconfirmed RBF"
transactions when creating new transactions: it simply adds the new outputs to the
existing txs (and updates the change).
The "RBF" flag is only enforced for unconfirmed txs, not for local txs.
The bug was that given a local LN funding tx, when creating a new channel
with "batch_rbf" enabled, we would modify the existing local tx, and
broadcast that.

related: #7298
2021-05-15 06:31:03 +02:00
SomberNight
9c1a51547a kivy: (trivial) clean-up imports in screens.py 2021-05-06 18:19:55 +02:00
SomberNight
3b0209a3a5 follow-up prev 2021-05-06 18:18:52 +02:00
ghost43
dd84acc109 Merge pull request #7234 from bitromortac/2104-invoice-amt-7184
invoice: fail gracefully with large amount
2021-05-06 15:43:36 +00:00
bitromortac
853e912885 invoice: fail gracefully with large amount 2021-05-06 15:37:17 +02:00
bitromortac
6ce96306ca util: check bip21 url for amount 2021-05-06 15:37:17 +02:00
SomberNight
c3ccfd7d19 android build: pin hashes of p4a recipes 2021-05-01 08:07:19 +02:00
SomberNight
5dcafaf082 mac build: link to issue in README 2021-05-01 06:40:28 +02:00
SomberNight
ef3293ab6a kivy: relocate atlas so it can be made a submodule later 2021-04-30 16:44:32 +02:00
jpph
0a56404f28 bip39 auto import, add coolwallet S weird derivation path (#7261)
auto import, add coolwallet S weird derivation path #7259

Would be nice to auto detect also coolwallet S derivation path.
In file bip39_wallet_formats.json :
{ "description": "coolwallet S derivation path using bip44 but with segwit script format",
"derivation_path": "m/44'/0'/0'",
"script_type": "p2wpkh-p2sh",
"iterate_accounts": true
}
see https://help.coolwallet.io/article/160-recover-btc-using-coolwallet-s-seed-without-the-wallet for their recovery instruction
2021-04-30 13:13:01 +00:00
ghost43
1e34b96725 Merge pull request #7258 from bitromortac/2104-openchannel-7219
kivy: improve openchannel dialog for trampoline
2021-04-30 11:41:27 +00:00
bitromortac
97b31880d7 kivy: improve openchannel dialog for trampoline 2021-04-30 13:32:07 +02:00
ThomasV
7789497140 release script: add username to signature file, upload files into airlock. 2021-04-30 10:27:33 +00:00
ThomasV
2b05615995 Do not perform webserver update from release script.
(see publish.sh and deploy.sh in the electrum-web repo)
2021-04-28 12:37:15 +00:00
SomberNight
1dfa81e7b6 contrib: fix find_restricted_dependencies for deps with version range
New release of pyinstaller (4.3) broke the script (which is used by freeze_packages.sh).

-----

Compare:

$ wget https://pypi.org/pypi/pyinstaller/4.3/json
$ cat json | jq ".info.requires_dist"
[
  "setuptools",
  "altgraph",
  "pyinstaller-hooks-contrib (>=2020.6)",
  "importlib-metadata ; python_version < \"3.8\"",
  "macholib (>=1.8) ; sys_platform == \"darwin\"",
  "pefile (>=2017.8.1) ; sys_platform == \"win32\"",
  "pywin32-ctypes (>=0.2.0) ; sys_platform == \"win32\"",
  "tinyaes (>=1.0.0) ; extra == 'encryption'",
  "pytest (>=2.7.3) ; extra == 'hook_testing'",
  "execnet (>=1.5.0) ; extra == 'hook_testing'",
  "psutil ; extra == 'hook_testing'"
]

$ wget https://pypi.org/pypi/pyinstaller/4.2/json | jq .
$ cat json | jq ".info.requires_dist"
null

$ wget https://pypi.org/pypi/qrcode/6.1/json
$ cat json | jq ".info.requires_dist"
[
  "six",
  "colorama ; platform_system == \"Windows\"",
  "tox ; extra == 'dev'",
  "pytest ; extra == 'dev'",
  "mock ; (python_version < \"3\") and extra == 'dev'",
  "zest.releaser[recommended] ; extra == 'maintainer'",
  "pillow ; extra == 'pil'",
  "pytest ; extra == 'test'",
  "pytest-cov ; extra == 'test'",
  "mock ; (python_version < \"3\") and extra == 'test'"
]
2021-04-26 20:45:40 +02:00
ghost43
de716ab07c Merge pull request #7247 from bitromortac/2104-fix-liquidity-hints
lnworker: fix path_finder access
2021-04-26 15:17:40 +00:00
SomberNight
8bba3b9ef5 network: also clear network.path_finder in stop_gossip
stop_gossip should undo start_gossip

related: #7242
2021-04-26 17:06:52 +02:00
ThomasV
04b1e8718a tx dialog (Qt): if possible, save psbt and fully signed transaction under the same basename. 2021-04-26 12:31:04 +02:00
bitromortac
4cb0b99996 lnworker: fix path_finder access 2021-04-26 08:43:50 +02:00
ThomasV
4411652b1c Merge pull request #7238 from SomberNight/202104_qt_send_max_tooltip
qt send tab: when clicking "Max", show tooltip explaining max amt
2021-04-23 20:22:53 +02:00
SomberNight
86ba37dc1c qt send tab: when clicking "Max", show tooltip explaining max amt
Beginner users often ask why they cannot send their full balance.
Hence, this intends to reduce support load.
In terms of UI, maybe there is a better way to do this but this was
easy to do and still a good first step IMHO.
2021-04-23 17:52:19 +02:00
SomberNight
37bde9baf9 qt: cpfp: fix handling "no dynamic fee estimates"
fixes #7237
2021-04-23 17:27:00 +02:00
ghost43
ecb083b283 Merge pull request #7231 from bitromortac/2104-fix-swap-check
swaps: fix swap sanity check
2021-04-23 12:57:44 +00:00
bitromortac
b97e51dbd8 swaps: fix off-by-one sanity check
Tolerates discrepancies in the swap amount crosschecks. To ensure we
calculate the send/receive amounts correctly we apply a check, using
amount inversion. The inversion is not exact up to +-1 due to used
floor and ceil functions in the methods. They are not invertible,
which is why we relax the check to off-by-ones.
2021-04-23 07:43:36 +02:00
SomberNight
1436760d3d qt coins tab: Ctrl+F now searches the whole prevout string
(not just the truncated string that is displayed in the list)
2021-04-22 20:56:00 +02:00
SomberNight
3b77340671 Qt MyTreeView: rm usages of Qt.UserRole, use explicit roles instead
This is a bit more verbose but it explicitly shows what data is being
used where.

Also rm implicitly setting editable_columns based on stretch_column.
2021-04-22 20:37:14 +02:00
SomberNight
720d60c391 (trivial) rm dead code
follow-up #7152
2021-04-22 17:57:50 +02:00
SomberNight
af44c846cc tests: add test for payto command 2021-04-22 17:13:56 +02:00
ghost43
0f83270053 Merge pull request #7152 from bitromortac/2103-liquidity-hints
Liquidity hints for pathfinding
2021-04-22 14:45:41 +00:00
SomberNight
bb381a263a android build: update p4a/buildozer to latest upstream
Rebased our branches on latest p4a develop HEAD and buildozer master HEAD.

Re `android.allow_backup = False` in buildozer.spec, now that this setting
has been exposed to buildozer.spec (see https://github.com/kivy/buildozer/pull/1206),
we could finally drop a commit from our branch that did the same:
7a7d415d90

Re commenting out `android.arch` in buildozer.spec, we are setting this
via the ENV VAR in make_apk.
2021-04-21 20:04:37 +02:00
ThomasV
8ce94248d4 kivy: update wizard message: pin->password 2021-04-21 10:33:11 +02:00