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

Expected behavior of GetStreamURI(RtspOverHttp ) with HTTP and HTTPS #70

Open
kumaakh opened this issue Feb 16, 2024 Discussed in #69 · 2 comments
Open

Expected behavior of GetStreamURI(RtspOverHttp ) with HTTP and HTTPS #70

kumaakh opened this issue Feb 16, 2024 Discussed in #69 · 2 comments

Comments

@kumaakh
Copy link

kumaakh commented Feb 16, 2024

Discussed in #69

Originally posted by kumaakh February 15, 2024
ONVIF TT version 23.06 rev 96
Compare the following two test cases:
5.1.1.2 MEDIA2 STREAMING – H.264 (RTP-Unicast/RTSP/HTTP/TCP)
5.1.1.15 MEDIA2 STREAMING – H.264 (RTP-Unicast/RTSP/HTTPS/TCP)

In both test cases ONVIFTT calls

GetStreamUri
wirh RtspOverHttp

In either case we return https://ip:443/ for the stream URI because the HTTP protocol has been disabled on our secure device we only allow HTTPS

5.1.1.15 passes fine but
5.1.1.2 fails at step 15 because ONVIT TT is trying to access an HTTPS URI without SSL handshake.
step 10,11,12,13 all pass, note step 13 chacks for URI scheme and finds it correct.

If you look at it from devices side we have no way of knowing why the client should be provided a non-SSL stream URI (http://) when the request came on a secure channel. Even If we do that, we are not secure any more.

I want to claim this as a bug on ONVIF TT implemetation as it is accessing a https:// URL without TLS handshake.

@madhurao68
Copy link
Member

We are investigating and revert with our response shortly. Thanks for your patience!

@madhurao68
Copy link
Member

Hello,
As a matter of fact, DTT doesn't support conformance over HTTPS. There are some tests which support HTTPS but not tests for conformance. Hope this helps!

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

No branches or pull requests

2 participants