spesmilo / spesmilo/electrum

NotADirectoryError: [Errno 20] Not a directory: '/Users/bryankoerner/.electrum/wallets/default_wallet/new.tmp.14118'

Open
#4,792 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug 🐞 topic-walletstorage 💾
Dominant language
Python
Stars
8.6k
Forks
3.5k
Avg merge
2d 8h
Merged PRs (30d)
61

Description

Crash Report

This crash report was reported through the automatic crash reporting system 🤖

Traceback

  File "electrum/util.py", line 882, in run_with_except_hook
  File "electrum/network.py", line 1085, in run
  File "electrum/network.py", line 1067, in wait_on_sockets
  File "electrum/network.py", line 719, in process_responses
  File "electrum/network.py", line 662, in process_response
  File "electrum/verifier.py", line 116, in verify_merkle
  File "electrum/address_synchronizer.py", line 437, in save_verified_tx
  File "electrum/util.py", line 336, in <lambda>
  File "electrum/util.py", line 332, in do_profile
  File "electrum/storage.py", line 107, in write
  File "electrum/storage.py", line 119, in _write

NotADirectoryError: [Errno 20] Not a directory: '/Users/bryankoerner/.electrum/wallets/default_wallet/new.tmp.14118'

Reporter

This issue was reported by 1 user(s):

Electrum Version Python Version Operating System Wallet Type Locale Stack
3.2.3 3.6.4 (default, Feb 28 2018, 03:24:19) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.42.1)] Darwin-14.5.0-x86_64-i386-64bit standard ?

Additional Information

Forgot password and recovered from a new wallet


Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading electrum/storage.py, especially write and _write, then trace the callers shown in verifier.py and address_synchronizer.py. Investigate why the temporary wallet path is treated as a directory and verify that wallet recovery and transaction verification no longer produce this NotADirectoryError.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.