search

Feathercoin-Foundation / electrum-ftc

Electrum-FTC; Feathercoin thin client. Follows upstream by rebasing FTC patches. Tags remain stable.
MIT License
10 stars 8 forks source link

Application crashes while uploading a file #25

Open Anilkumar18 opened 6 years ago

Anilkumar18 commented 6 years ago

Where ever there is an option to load a file externally (read file option) , the system accepts only .txt and .csv files. Application crashes with all other extension files(like .doc,.jpeg,.xls etc)

read_file_icon_error

Steps to replicate the issue[This is one of the scenario]: -> In install wizard -> Enter Wallet name ->Select standard Wallet ->Select Create a new seed -> Select standard -> Copy the seed value from the next screen and save it in to a doc file -> In the next screen when we try to upload the doc file using the read file icon the application crashes.

Note: Screenshot of the icon is attached

lclc commented 6 years ago

Thanks for reporting. Please always use the suggested style with the pre-written text when creating a new GitHub issue so we know which version you use, what OS, etc.

Please check if this bug also exist in Electrum (BTC).

If you say it accepts .csv, what's the content of that CSV file you used for testing? I'd expect the only thing that works is a plaintext file with only the seed words in.

Anilkumar18 commented 6 years ago

1)Please check if this bug also exist in Electrum (BTC). In Electrum BTC there is no option to load a file externally in the confirm seed window. In Send tab, Pay to field there is an option to load file externally. Here it accepts text files but when we upload any other extension files it throws an error "Error opening file: 'charmap' codec can't decode byte 0x81 in position 18: character maps to "

2)If you say it accepts .csv, what's the content of that CSV file you used for testing? We generally copy the seed as such to a .csv file /.txt file and save . But when we do the same with .doc/.xls file or any other extensions it crashes.

Expected Behavior : Either a message should be displayed on what type of file would be accepted or an error message should be thrown.

Tested on Windows 8.1,10