Title: Dataikuapi is not correctly honoring "DKU_NO_CHECK_CERTIFICATE" in requests · Issue #243 · dataiku/dataiku-api-client-python · GitHub
Open Graph Title: Dataikuapi is not correctly honoring "DKU_NO_CHECK_CERTIFICATE" in requests · Issue #243 · dataiku/dataiku-api-client-python
X Title: Dataikuapi is not correctly honoring "DKU_NO_CHECK_CERTIFICATE" in requests · Issue #243 · dataiku/dataiku-api-client-python
Description: In theory, Dataiku supports using self-signed certificates by setting e.g. using an environmental variable DKU_NO_CHECK_CERTIFICATE or by setting no_check_certificate=True in the set_remote function call of the non-open source dataiku pa...
Open Graph Description: In theory, Dataiku supports using self-signed certificates by setting e.g. using an environmental variable DKU_NO_CHECK_CERTIFICATE or by setting no_check_certificate=True in the set_remote functio...
X Description: In theory, Dataiku supports using self-signed certificates by setting e.g. using an environmental variable DKU_NO_CHECK_CERTIFICATE or by setting no_check_certificate=True in the set_remote functio...
Opengraph URL: https://github.com/dataiku/dataiku-api-client-python/issues/243
X: @github
Domain: patch-diff.githubusercontent.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Dataikuapi is not correctly honoring \"DKU_NO_CHECK_CERTIFICATE\" in requests","articleBody":"In theory, Dataiku supports using self-signed certificates by setting e.g. using an environmental variable DKU_NO_CHECK_CERTIFICATE or by setting `no_check_certificate=True` in the `set_remote` function call of the non-open source dataiku package.\r\n\r\nHowever, this parameter is not passed to the api_client(), which causes requests to fail which are made to Dataiku instances using self-signed certificates. It is possible to mitigate this by setting the verify value of the internal Session object to false:\r\n\r\n client = dataiku.api_client()\r\n client._session.verify = False\r\n\r\nWhich is not obvious and requires looking through the internals of the package.\r\n\r\nA fix at the central call site of the dataikuapi package, `_perform_http`:\r\nhttps://github.com/dataiku/dataiku-api-client-python/blob/fc51e29a12a1f50717771522e1b450ae91ff4d87/dataikuapi/dssclient.py#L1228-L1233\r\nwould only be possible in parts by duplicating the logic to check the \"DKU_NO_CHECK_CERTIFICATE\" variable or config files, but would still fail for Dataiku session set up by using the `set_remote_dss` function call.\r\n\r\nThe best option would be to add a \"no_check_certificate\" parameter to the `DSSClient` class that can be set when creating an instance of the class. But as the `DSSClient` is also used by the closed-source `dataiku` package, it seems to me that this can't be solved by contributing to the open source `dataikuapi` package.\r\n","author":{"url":"https://github.com/seb-ehm","@type":"Person","name":"seb-ehm"},"datePublished":"2022-06-17T15:52:32.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/243/dataiku-api-client-python/issues/243"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:f0c83d16-9db5-fcd5-ab8b-c51e7f243348 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | D448:1F64D6:1022276:145BBB7:69916CDD |
| html-safe-nonce | 3da1f030adceae520318eead046ac733f4057d09a4dc4bb0886c8543ed014689 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJENDQ4OjFGNjRENjoxMDIyMjc2OjE0NUJCQjc6Njk5MTZDREQiLCJ2aXNpdG9yX2lkIjoiODI3NDI0NjgyNTQ2MzkyMjg1IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 8990030bbd2a0a14393b6dabd71af099acfba2f64ac344f42f536bf9626239dd |
| hovercard-subject-tag | issue:1275205286 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/dataiku/dataiku-api-client-python/243/issue_layout |
| twitter:image | https://opengraph.githubassets.com/b42b1a5c08250f2081f6ada12c0b2b5e15ac94902ca3cdfdf996c1a3d1234076/dataiku/dataiku-api-client-python/issues/243 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/b42b1a5c08250f2081f6ada12c0b2b5e15ac94902ca3cdfdf996c1a3d1234076/dataiku/dataiku-api-client-python/issues/243 |
| og:image:alt | In theory, Dataiku supports using self-signed certificates by setting e.g. using an environmental variable DKU_NO_CHECK_CERTIFICATE or by setting no_check_certificate=True in the set_remote functio... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | seb-ehm |
| hostname | github.com |
| expected-hostname | github.com |
| None | 42c603b9d642c4a9065a51770f75e5e27132fef0e858607f5c9cb7e422831a7b |
| turbo-cache-control | no-preview |
| go-import | github.com/dataiku/dataiku-api-client-python git https://github.com/dataiku/dataiku-api-client-python.git |
| octolytics-dimension-user_id | 2335170 |
| octolytics-dimension-user_login | dataiku |
| octolytics-dimension-repository_id | 40004578 |
| octolytics-dimension-repository_nwo | dataiku/dataiku-api-client-python |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 40004578 |
| octolytics-dimension-repository_network_root_nwo | dataiku/dataiku-api-client-python |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 848bc6032dcc93a9a7301dcc3f379a72ba13b96e |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width