Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when sending broadcast text message but choosing image file #264

Closed
KP4AJ opened this issue Oct 22, 2023 · 2 comments
Closed

Error when sending broadcast text message but choosing image file #264

KP4AJ opened this issue Oct 22, 2023 · 2 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@KP4AJ
Copy link

KP4AJ commented Oct 22, 2023

Hi:
This when trying to broadcast text file but sending image (jpeg). Windows 10 D-Rats latest git code. Obviously if user choose correctly this problem should not occur. Here the traceback:
UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 65: character maps to

Win10 Git Code Broadcast image file:
Traceback (most recent call last):

File "C:/Users/edfel/D-Rats/d_rats/ui/main_chat.py", line 876, in _bcast_file
data = file_handle.read()
^^^^^^^^^^^^^^^^^^

File "C:/msys64/mingw64/lib/python3.11/encodings/cp1252.py", line 23, in decode
return codecs.charmap_decode(input,self.errors,decoding_table)[0]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 65: character maps to

73'
Edfel
KP4AJ

@wb8tyw wb8tyw added the bug Something isn't working label Jun 9, 2024
@wb8tyw
Copy link
Contributor

wb8tyw commented Jun 9, 2024

I guess that D-Rats should catch the exception and just log (hopefully only once) that the file can not be sent as it has non-text characters. However as with the other ticket, there is no way to know if a text file is displayable on another platform.

@wb8tyw wb8tyw added the duplicate This issue or pull request already exists label Jun 9, 2024
@wb8tyw
Copy link
Contributor

wb8tyw commented Jun 9, 2024

This is a duplicate of #272

@wb8tyw wb8tyw closed this as completed Jun 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants