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

missing https certificate throws exception #57

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GsDublin
Copy link

With https function throws:
SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate',...

With https function throws:
SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate',...
@Benehiko
Copy link
Member

Hi @GsDublin
Thank you for your contribution :)

I think we need to have the default set to True and allow people to pass down the value when opening a stream.

@GsDublin
Copy link
Author

Hello @Benehiko,
Iam a new git member and iam trying to use this tool in both directions ;-)

I think it should be a global parameter.
Im sure the problem would be at each call, of each .get function.
Should i try to do the changes, and you check my modifications like this?

Some people deactivated the https, some people use only https, and someone uses own certificates?

Iam using following cams:
E1 Zoom, RLC-810A, RLC-511, RLC-410-5MP, RLC-810A (IPC_523128M8MP) and RLC-820A (IPC_523128M8MP)

For the "new" one (e.g. RLC-810A, RLC-820A) we need to add a new API command: GetRecV20, GetFtpV20, GetPushV20.

@Benehiko
Copy link
Member

HI @GsDublin no worries :)

You can use the kwargs param from here https://github.com/ReolinkCameraAPI/reolinkapipy/blob/master/reolinkapi/camera.py#L6-L12

and pass along an expected value for verifying ssl and if it is true check for passed down ssl certificates
like we do here

Request.proxies = kwargs.get("proxy") # Defaults to None if key isn't found

then you can call it through the classself. property like we do here

ip=self.ip, username=self.username, password=self.password, profile=self.profile, proxies=proxies, callback=callback)

For the other API functions I would recommend opening separate PRs so it is easier to keep track of changes :)

@Benehiko
Copy link
Member

Hi @GsDublin any update on this?

@Benehiko Benehiko marked this pull request as draft April 17, 2022 11:02
@Benehiko Benehiko self-requested a review April 17, 2022 11:02
@Benehiko Benehiko added the enhancement New feature or request label Apr 17, 2022
@Vaelatern
Copy link

Vaelatern commented Nov 8, 2022

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).

@Vaelatern
Copy link

@Benehiko have a look at this commit? Vaelatern@8cf5f7f
I decided not to proceed with it, but it might be useful

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants