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

support self signed certificates #25

Open
maddanio opened this issue Feb 26, 2024 · 9 comments
Open

support self signed certificates #25

maddanio opened this issue Feb 26, 2024 · 9 comments

Comments

@maddanio
Copy link

Hello,

I need to use this wonderful plugin against a server of ours inside a vpn which has a self signed cert. so I would need to tell the plugin the cert or disable ssl checking, either one would be good for me, any hints how to do that?

Best

@yeraydiazdiaz
Copy link
Owner

Hi @maddanio, I'd rather not disable certificate validation if we can help it as it's generally a bad idea. But now that we have settings since 1.0.0 we could add an option for users to set options we can pass through to urllib3.

@maddanio
Copy link
Author

Of course not by default, ideally per request explicitly. We just need it in a rather peculiar setup. Would be super great as it seems to be stupidly hard to find a tool for this that does not try to get you into some weird enterprise subscription. So are you saying i can do this right now?

@yeraydiazdiaz
Copy link
Owner

No, sorry, I was describing how it could be implemented.

@maddanio
Copy link
Author

well, per query would be great, global would also be ok. i can also try my hands at a pr, but not sure I will find the time :(

@maddanio
Copy link
Author

maddanio commented Mar 10, 2024

as a syntax I would suggest @@ as a prefix as additional urlopen arguments, or @urlopen.<keyword>. I can make a pr

@maddanio
Copy link
Author

ah, I just noticed the argument for certificate validation is on the connection pool level. so I guess a setting would be better

@maddanio
Copy link
Author

ok, I added it somehow, but the settings dont appear in the sublime settings list? also I am not sure if reload works

@maddanio
Copy link
Author

#29

@maddanio
Copy link
Author

aleternatively/additionally it might be better to allow additional certificates?

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