Disabled SSL context

[agajdosi]

Updater disables SSL, for example here: https://github.com/CGCookie/blender-addon-updater/blob/master/addon_updater.py#L672

Wouldn't it be safer to set the context?

context = ssl.create_default_context(purpose=ssl.Purpose.CLIENT_AUTH)
context.load_verify_locations(certifi.where())

Is there any reason why it is disabled? Is it good idea to enable it in our project, if we want security?

[TheDuckCow]

This was a change integrated after proposed by someone else using the repository, citing issues with it working in a studio setting where it was disabled. It was a bit of a niche case, but we opted for the change to increase the usability. I'm happy to revisit the point though, just need to weigh in the consequences and implications.

If you use this modified line yourself, have you found things working still as you expect generally?

[TheDuckCow]

Edit to add more context, this is where the issue arose: #35 (comment)

I could also be recalling a separate email chain that wasn't on github, but probably makes the most sense to go off of the situation described here. Let me know if you have any thoughts on that, otherwise I'll try to get back to this to read some more and see what the right thing to do is.