fix: use stdout instead of stderr for reading rpi-connect process out…

…put - closes #174

CHANGELOG.md

@@ -9,6 +9,7 @@
 - refactor(core): rerender screen when rendering on the display is resumed like when returning from viewfinder to avoid artifacts
 - refactor(docker): make composition menus responsive: showing spinner, etc
 - fix: pass raw bytes to `DisplayRenderEvent` and `DisplayCompressedRenderEvent` to avoid encoding issues
+- fix: use stdout instead of stderr for reading rpi-connect process output - closes #174
 
 ## Version 1.1.0
 

ubo_app/services/050-rpi-connect/sign_in_page.py

@@ -48,12 +48,12 @@ async def login(self: SignInPage) -> None:
                 '/usr/bin/env',
                 'rpi-connect',
                 'signin',
-                stdout=subprocess.DEVNULL,
-                stderr=subprocess.PIPE,
+                stdout=subprocess.PIPE,
+                stderr=subprocess.DEVNULL,
             )
-            if self.process.stderr is None:
+            if self.process.stdout is None:
                 return
-            output = (await self.process.stderr.readline()).decode()
+            output = (await self.process.stdout.readline()).decode()
             regex = r'^Complete sign in by visiting (?P<url>[^\n]*)'
             match = re.search(regex, output)
             if match: