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

ToxProxy AppImage generates (apparently) invalid ToxIDs #239

Closed
geebotron opened this issue Feb 24, 2021 · 13 comments
Closed

ToxProxy AppImage generates (apparently) invalid ToxIDs #239

geebotron opened this issue Feb 24, 2021 · 13 comments
Labels

Comments

@geebotron
Copy link

geebotron commented Feb 24, 2021

Describe the bug
The output of running the ToxProxy AppImage produces a ToxID, that when pasted into TRIfA reports that the ID is invalid.

To Reproduce
Steps to reproduce the behavior:
I ran the AppImage on a Debian stable box.

Subsequently, I deleted the files the AppImage produced in the directory it lives in, and ran it again (as new) to produce a different Tox ID. This new ID was still invalid, according to TRIfA.

Expected behavior
Pasting the generated ToxID into the "Add Friend" dialog would result in the ToxProxy ID being accepted.

Your system information:

  • TRIfA Version: 1.0.94
  • ToxProxy Version: 0.99.4
  • App from: F-Droid
  • Android OS Version: 11
  • Device: Sansung S10e

Add screenshots, if possible

Additional context
This may be a problem with TRIfA itself - as pasting the IDs into another Tox client seemed to accept them. They appear to be 76 chars long, and with no unusual chars ...
Strangely - deleting the last character of the generated ToxID allowed the ID to be claimed as valid in TRIfA - but it still didn't add a friend or work correctly.
Could this be a validation issue in TRIfA?

Also - Regarding firewalls:
Is it required to make firewall adjustments to allow incoming connections on the host of the AppImage for UDP on port 33445? I noticed it was listening on port 33445 so I allowed incoming cxns in the firewall rules.

Thanks

@geebotron geebotron added the bug label Feb 24, 2021
@zoff99
Copy link
Owner

zoff99 commented Feb 24, 2021

Is it required to make firewall adjustments to allow incoming connections on the host of the AppImage
--> no

@zoff99
Copy link
Owner

zoff99 commented Feb 24, 2021

can you post a screenshot of trifa when it says the ID is invalid?
do you have any "spaces" or other characters in front or at the end of the ToxID?
like "tox:" at start or something?

@zoff99
Copy link
Owner

zoff99 commented Feb 24, 2021

just tried it again on the emulator, and it works fine.
do not copy the "ToxID:" part.

if it's still not working, please make screenshots and post more details

@geebotron
Copy link
Author

I only capture the part after the "ToxID:" and I have pasted the same string into QTox which seemed happy.

ToxProxy version: 0.99.4
#############################################################
#############################################################

ToxID:C012744F65AC1F40F70920F808506E7E54DB943578FA55652009BB610B87F32E672F6E81F870

#############################################################
#############################################################

Screenshot_20210224-190307_TRIfA

@zoff99
Copy link
Owner

zoff99 commented Feb 24, 2021

can you scroll the text fully to the left, to show the start of the pasted ToxID please?

@geebotron
Copy link
Author

Screenshot_20210224-192231_TRIfA

@zoff99
Copy link
Owner

zoff99 commented Feb 24, 2021

there must be something wrong with the text you copied.
if (editable.length() == (TOX_ADDRESS_SIZE * 2))
it only checks the length of the input text

@zoff99
Copy link
Owner

zoff99 commented Feb 24, 2021

did you copy the final "\n" or something?
can you make sure the pasted text is TEXT only no formatting or special chars?

maybe remove the front letter and type it in manually and remove the end letter and type that in manually.
i would need to know where the problem is, to fix it.

can you join the matrix channel?

@geebotron
Copy link
Author

Yeah. It's like there's some sort of invisible char captured. If I delete the final char of the string (or any single char) the "incorrect ToxID" message goes away.
I'm damned if I can find any additional chars - I pasted the clipboard with the ToxID into a plain text editor. Nothing unusual at the start or the end. Weird.

@geebotron
Copy link
Author

Just joined the Matrix channel BTW ...

@zoff99
Copy link
Owner

zoff99 commented Feb 24, 2021

it can only be the "\r" or "\n" when you select the line ending when you copy.
can you paste into a file and display in hex?

@zoff99 zoff99 closed this as completed in e765b9b Feb 24, 2021
@zoff99 zoff99 reopened this Feb 24, 2021
@zoff99
Copy link
Owner

zoff99 commented Mar 14, 2021

note to self: remove any \r and \n in pasted ToxIDs (if not already done)

@zoff99
Copy link
Owner

zoff99 commented Jun 22, 2021

should be fixed by e765b9b
lets close it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants