TermySequence / termysequence

Main source distribution, containing termy-server and qtermy
https://termysequence.io
GNU General Public License v2.0
16 stars 2 forks source link

Case sensitivity issue with files tool #6

Closed genrtul closed 5 years ago

genrtul commented 5 years ago

I'm using version 1.1.4 from the Fedora 28 repos. When I try to download files with the files tool I get an error popup "The file's host 'max' does not match the terminal's host 'MAX'". The correct hostname is the upcased version, as reported by the "hostname" command. The issue persists on all hosts I control (they all have upcased names)

sigalrm commented 5 years ago

Thanks for this report. I'll make the comparison case-insensitive. The QUrl.host() getter is returning a lowercase name which appears to be the cause of this issue.

sigalrm commented 5 years ago

A patched termy-qt package is now available in COPR for testing.

genrtul commented 5 years ago

The issue is resolved in the patched version

sigalrm commented 5 years ago

Updated packages are now available for Ubuntu, OpenSUSE, and Arch. Updates submitted for F28+.

sigalrm commented 5 years ago

Fedora update is in stable and will be available once it hits the mirrors.