-
Notifications
You must be signed in to change notification settings - Fork 48
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
missing https certificate throws exception #57
base: master
Are you sure you want to change the base?
Conversation
With https function throws: SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate',...
Hi @GsDublin I think we need to have the default set to True and allow people to pass down the value when opening a stream. |
Hello @Benehiko, I think it should be a global parameter. Some people deactivated the https, some people use only https, and someone uses own certificates? Iam using following cams: For the "new" one (e.g. RLC-810A, RLC-820A) we need to add a new API command: GetRecV20, GetFtpV20, GetPushV20. |
HI @GsDublin no worries :) You can use the and pass along an expected value for verifying ssl and if it is true check for passed down ssl certificates
then you can call it through the class reolinkapipy/reolinkapi/mixins/stream.py Line 26 in 8b6049b
For the other API functions I would recommend opening separate PRs so it is easier to keep track of changes :) |
Hi @GsDublin any update on this? |
I'm expecting I'll pick this up just to clean up my log files while I'm using this (with the changes to pass it down). |
@Benehiko have a look at this commit? Vaelatern@8cf5f7f |
With https function throws:
SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate',...