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

Fix onvif source: invalid control character in URL #662

Closed
boerti71 opened this issue Sep 30, 2023 · 5 comments
Closed

Fix onvif source: invalid control character in URL #662

boerti71 opened this issue Sep 30, 2023 · 5 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists
Milestone

Comments

@boerti71
Copy link

boerti71 commented Sep 30, 2023

Hej!
I'm using Home assistant:
Home Assistant 2023.9.3
Supervisor 2023.09.2
Operating System 10.5
Frontend 20230911.0 - latest

I also edited the go2rtc.yaml file in Home assistants file editor:

streams:
  entre: onvif://admin:xxxxxxx@192.168.1.72:80

I also tried with

onvif://adminxxxxxx@192.168.1.72:80?subtype=MainStreamToken

but I alway get the Invalind character error.

here is the log...

17:39:09.132 INF go2rtc version 1.7.1 linux/amd64
17:39:09.132 INF [api] listen addr=0.0.0.0:1984
17:39:09.136 INF [rtsp] listen addr=0.0.0.0:8554
17:39:09.136 INF [webrtc] listen addr=0.0.0.0:8555/tcp
17:39:33.764 ERR github.com/AlexxIT/go2rtc/internal/mjpeg/init.go:166 > error="streams: parse "\r\n": net/url: invalid control character in URL"

any ideas?

Thanks!

@AlexxIT AlexxIT added the bug Something isn't working label Oct 5, 2023
@AlexxIT AlexxIT added this to the v1.7.2 milestone Oct 5, 2023
@AlexxIT
Copy link
Owner

AlexxIT commented Oct 5, 2023

It will be easier if you can provide access to your stream (80 port)
https://github.com/AlexxIT/go2rtc/wiki/Tunnel-RTSP-camera-to-Intenet

@AlexxIT AlexxIT changed the title invalid control character in URL Fix onvif source: invalid control character in URL Oct 11, 2023
@AlexxIT AlexxIT added the duplicate This issue or pull request already exists label Oct 11, 2023
@AlexxIT
Copy link
Owner

AlexxIT commented Oct 11, 2023

#638

@AlexxIT
Copy link
Owner

AlexxIT commented Oct 13, 2023

@boerti71 have you tested fix?

@boerti71
Copy link
Author

I'm sorry, but i installed in the meantime Frigate on an raspi 4...

@AlexxIT
Copy link
Owner

AlexxIT commented Oct 14, 2023

felipecrs pushed a commit to felipecrs/go2rtc that referenced this issue Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants