-
Notifications
You must be signed in to change notification settings - Fork 775
New feature Onvif PTZ controls on screen #1665
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
base: master
Are you sure you want to change the base?
Conversation
|
if someone would like to play, compiled images are here: example config: at the moment the onvif PTZ controls will be displayed on the page if the name of the stream ends with " |
|
Hey @notjulian I like your idea for ptz. But I have a problem with this branch, when I want to send a request from a custom website in Chrome, I get this erros:
I have set this in the go2rtc configuration:
Could it be that the CORS handling from the api code does not work here? |
It could be, I haven't tried yet ... Does it work if you connect directly to the IP? I'll let you know ... |
|
Interestingly, it works under Firefox or when I send the request directly. |
|
I also use your Docker image and get the same result when I use the integrated website with the name extension “_ptz” under Chrome. But when I use my own website in Chrome, the ptz request does not work. Under Firefox or direct requests everything works fine. Maybe I'm doing something wrong, but I don't know where. The only problem in my case is Chrome. |
|
I found out that the preflights use “OPTIONS” methods which are not allowed in your ptzHandler. |
|
when is enabled it should do L216 from the log
can you please post yours? |
|
@mchilli can you please try with the latest image ... thanks |
|
Oh yes, now it works on all my devices and browsers! Thank you so much for your quick help! |
Convention over configuration at the moment So the src must contains _ptz to enable the controls
… unit tests for fragment parsing
…vif and update unit tests for fragment parsing
|
I would love for something like this to be merged. Is there anything the community can do to speed this process? |




Hi Alex,
Thanks for this amazing project!
I'd like to have the PTZ controls...
So I made some changes. I understand that more features means more code to maintain and more bugs, so I tried to be as minimal as possible. Also, the layout maybe needs a UI/UX review.
Please let me know what you think.
If it needs improvement, I'm happy to make changes.
PS: at the moment is only ptz, no zoom
PPS: to display the controls the src needs to have the _ptz in the name (happy to improve ...)