search

inspera / blackbricks

Black for Databricks notebooks
MIT License
44 stars 9 forks source link

Added handling for verify_ssl #28

Closed msmall318 closed 2 years ago

msmall318 commented 2 years ago

The current blackbricks doesn't handle the databricks cli insecure argument. This pull request enables handling. See discussion at the community forum. This change incorporates the solution Megan05 mentioned in same thread. Capture

bsamseth commented 2 years ago

Thanks for adding this! I slightly modified your initial commit to account for the slightly odd way the databricks CLI handles the insecure option. They treat any value as the option being enabled, even if you set it to false. Even though that might be a little unintuitive, blackbricks should give it the same treatment.

I just figured I would add a link to a possible solution to the SSL issues I'm guessing you are facing. Just in case it helps: https://stackoverflow.com/a/73650193/3377926

msmall318 commented 2 years ago

Thanks! I'll look into that option, too.