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
Comments
mrlt8
added a commit
that referenced
this issue
May 4, 2024
hmmm, I think you're right. could you try the latest dev branch? |
Cloned dev branch this morning. Can confirm that it's working. Thanks! |
Merged
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
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 simplyos.mkfifo(self.FIFO)
?The text was updated successfully, but these errors were encountered: