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

Fix pairing UI on Linux #47

Closed
wants to merge 1 commit into from
Closed

Conversation

VShell
Copy link

@VShell VShell commented Sep 7, 2023

On Linux when building with Qt5, the pairing UI fails to display any devices, with the following warnings:

default       2023.09.07 17:11:32.358 10950 W (unknown:0)                                                                       : QMetaProperty::read: Unable to handle unregistered datatype 'RemoteDeviceFilterModel*' for property 'governikus::RemoteServiceModel::availableDevicesInPairingMode'
default       2023.09.07 17:11:32.358 10950 W (unknown:0)                                                                       : QMetaProperty::read: Unable to handle unregistered datatype 'RemoteDeviceFilterModel*' for property 'governikus::RemoteServiceModel::unavailablePairedDevices'
default       2023.09.07 17:11:32.359 10950 W (unknown:0)                                                                       : QMetaProperty::read: Unable to handle unregistered datatype 'RemoteDeviceFilterModel*' for property 'governikus::RemoteServiceModel::availablePairedDevices'

After registering this class with qRegisterMetaType, the pairing UI functions correctly on Linux, allowing me to pair a smartphone and authenticate to service providers using my Aufenthaltstitel.

@CLAassistant
Copy link

CLAassistant commented Sep 7, 2023

CLA assistant check
All committers have signed the CLA.

@misery
Copy link
Contributor

misery commented Sep 8, 2023

Interesting... we have no problems on tested Linux-Distributions. Which distro do you use? Did you use a distro package or self built?

@VShell
Copy link
Author

VShell commented Sep 8, 2023

I originally found this issue with the NixOS package, with no patches/etc. Specific version info dumped from the application:

init          2023.09.08 10:12:09.139 82874 I printInfo(init/Bootstrap.cpp:60)                                           : ### Application: AusweisApp2
init          2023.09.08 10:12:09.139 82874 I printInfo(init/Bootstrap.cpp:60)                                           : ### Application Version: 1.26.7
init          2023.09.08 10:12:09.139 82874 I printInfo(init/Bootstrap.cpp:60)                                           : ### Organization:
init          2023.09.08 10:12:09.139 82874 I printInfo(init/Bootstrap.cpp:60)                                           : ### Organization Domain:
init          2023.09.08 10:12:09.139 82874 I printInfo(init/Bootstrap.cpp:60)                                           : ### System: NixOS 23.11 (Tapir)
init          2023.09.08 10:12:09.139 82874 I printInfo(init/Bootstrap.cpp:60)                                           : ### Kernel: 6.1.51
init          2023.09.08 10:12:09.139 82874 I printInfo(init/Bootstrap.cpp:60)                                           : ### Architecture: x86_64
init          2023.09.08 10:12:09.139 82874 I printInfo(init/Bootstrap.cpp:60)                                           : ### Device: katy
init          2023.09.08 10:12:09.139 82874 I printInfo(init/Bootstrap.cpp:60)                                           : ### Qt Version: 5.15.10
init          2023.09.08 10:12:09.139 82874 I printInfo(init/Bootstrap.cpp:60)                                           : ### OpenSSL Version: OpenSSL 3.0.10 1 Aug 2023

@mptei
Copy link

mptei commented Sep 8, 2023

I have the same problem with AusweisApp2-1.26.7 on Gentoo-Linux and on Endeavour.
The same warning appears and the mobile phone is not recognized.
After applying the PR it works (on Gentoo).

##################################################                                     
### Application: AusweisApp2                                                           
### Application Version: 1.26.7                                                        
### Organization:                                                                      
### Organization Domain:                                                               
### System: Gentoo Linux                                                               
### Kernel: 6.1.41-gentoo                                                              
### Architecture: x86_64                                                               
### Device: bigmon                                                                     
### Qt Version: 5.15.10                                                                
### OpenSSL Version: OpenSSL 3.0.9 30 May 2023                                         
################################################## 

@misery
Copy link
Contributor

misery commented Sep 20, 2023

Is it possible to try with Qt 6.4 or higher to isolate the problem?

@misery
Copy link
Contributor

misery commented Oct 23, 2023

@VShell @mptei Could you retry with Qt 6.4?

@mptei
Copy link

mptei commented Oct 24, 2023

I have tried it on another computer:

##################################################
### Application: AusweisApp2
### Application Version: 1.26.7
### Organization: 
### Organization Domain: 
### System: EndeavourOS
### Kernel: 6.5.7-arch1-1
### Architecture: x86_64
### Device: mikesair
### Qt Version: 6.6.0
### OpenSSL Version: OpenSSL 3.1.3 19 Sep 2023
##################################################

it shows the same behavior (not finding the smartphone), but I can't see the warning from #47 (comment).

@misery
Copy link
Contributor

misery commented Oct 25, 2023

@mptei Is it fixed with this patch?

@sleepylinde
Copy link

I'm assuming this has not been fixed, since I downloaded the package today from the AUR and the app does not find my phone to pair it.

##################################################
### Application: AusweisApp
### Application Version: 2.0.0
### Organization: 
### Organization Domain: 
### System: EndeavourOS
### Kernel: 6.6.8-zen1-1-zen
### Architecture: x86_64
### Device: linde
### Qt Version: 6.6.1
### OpenSSL Version: OpenSSL 3.2.0 23 Nov 2023

Works fine on a windows pc in the same network.

@misery
Copy link
Contributor

misery commented Jan 2, 2024

I'm assuming this has not been fixed, since I downloaded the package today from the AUR and the app does not find my phone to pair it.

Do you still have Unable to handle unregistered datatype 'RemoteDeviceFilterModel*' in your log file? Please provide a log.

@sleepylinde
Copy link

It just does not have any errors regarding the scan (just a couple about images and failed downloads of what I'm assuming resources)

I opened the pairing dialog, waited a solid minute and then switched to another tab and then back to pairing (twice) while also having the pairing open on my smartphone.
error_log.log

@kgraefe
Copy link

kgraefe commented Jan 12, 2024

Not sure if that's helpful to anyone, but I thought I have the same problem. I'm fairly new to EndeavourOS and I did not know it has a firewall running. Opening UDP port 24727 in the firewall-config GUI solved it for me on

##################################################
### Application: AusweisApp
### Application Version: 2.0.0
### Organization:
### Organization Domain:
### System: EndeavourOS
### Kernel: 6.6.10-arch1-1
### Architecture: x86_64
### Device: okeanos
### Qt Version: 6.6.1
### OpenSSL Version: OpenSSL 3.2.0 23 Nov 2023
##################################################

@misery
Copy link
Contributor

misery commented Jan 23, 2024

Seems this error in log file in not reproducible in Qt6. Feel free to re-open if it has the SAME log messages.

@misery misery closed this Jan 23, 2024
@Nols1000
Copy link

Hey I had a similar issue on my openSuse machine. Opening the firewall port did allow pairing the device. Is there some documentation on this, somewhere other than this PR?

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

Successfully merging this pull request may close these issues.

7 participants