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

mtx_event.py breaks for non-root user #1174

Closed
GiZZoR opened this issue May 3, 2024 · 2 comments · Fixed by #1184
Closed

mtx_event.py breaks for non-root user #1174

GiZZoR opened this issue May 3, 2024 · 2 comments · Fixed by #1184

Comments

@GiZZoR
Copy link

GiZZoR commented May 3, 2024

I was trying to deploy wyze-bridge manually to a LXC container in proxmox, and noticed that when I tried to run the app as a non-root user I kept getting this error:
May 03 15:41:49 wyze-bridge gunicorn[3695]: File "/app/wyzebridge/mtx_event.py", line 56, in open_pipe
May 03 15:41:49 wyze-bridge gunicorn[3695]: self.pipe_fd = os.open(self.FIFO, os.O_RDWR | os.O_NONBLOCK)
May 03 15:41:49 wyze-bridge gunicorn[3695]: ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
May 03 15:41:49 wyze-bridge gunicorn[3695]: PermissionError: [Errno 13] Permission denied: '/tmp/mtx_event'

I might be wrong, but I think the usage of os.mkfifo here seems incorrect. As far as I can tell, os.mkfifo expects a "mode" and not the "flags" from os.open.

Is there some reason for not changing os.mkfifo(self.FIFO, os.O_RDWR | os.O_NONBLOCK) to simply os.mkfifo(self.FIFO)?

mrlt8 added a commit that referenced this issue May 4, 2024
@mrlt8
Copy link
Owner

mrlt8 commented May 4, 2024

hmmm, I think you're right. could you try the latest dev branch?

@GiZZoR
Copy link
Author

GiZZoR commented May 6, 2024

Cloned dev branch this morning. Can confirm that it's working. Thanks!

@mrlt8 mrlt8 linked a pull request May 12, 2024 that will close this issue
mrlt8 added a commit that referenced this issue May 12, 2024
* Sign motion event request #1125

* use device id instead of mac #1125

* remove wrong flags from mkfifo #1174

* less aggressive flush #1159 #1167

* Use K10052 for setting FPS #1161

* Refactor bits for going above 255 for quality

* Revert ffmpeg changes #1159 #1167

* version is obsolete

* default quality to hd180

* Update Wyze iOS App version from v2.44.5.3 to v2.50.6.1 (#1176)

* use struct pack

* Token based auth over the webUI

* Fix redirect for Home Assistant Ingress

* Use request headers to fix redirect for HA

* Change GET to POST for webhooks data

* Deprecate ifttt_webhook in favor of webhooks

* use yml for HA config and make credentials optional

* keep trying to identify audio #1172

* Snapshot on motion and push to mqtt #709 #970

* Add event time to motion message

* refactor auth

* EVENT_API option #1125

* Add additional headers  #1125

* Audio sync with higher bitrate

* Debug api request #1125

* Update api.py

* only debug on error

* Tweak audio sync

* don't raise error on lost frame

* clear buffer if out of sync

* Unique macs only #1125

Co-Authored-By: Cameron <32912464+kiwi-cam@users.noreply.github.com>

* Require auth by default and block non-ingress access #1181

* Allow non-ingress access with auth #1181

* Remove retain flag from commands #1182

* update webrtc to work with auth streams

* Add WB_API and rename WEB to WB #1181

WEB_USERNAME > WB_USERNAME
WEB_PASSWORD > WB_PASSWORD

* HA move /config/wyze-bridge/ to /config/

* Don't notify substream event and remove v2 #1125

* WebUI Auth related config for HA

* Don't retain discovery message? #1182

* Case sensitive credentials for WebUI

* changelog and readme

---------

Co-authored-by: Cameron <32912464+kiwi-cam@users.noreply.github.com>
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 a pull request may close this issue.

2 participants