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

[CM-9020] add config for disabling TLS verification #107

Conversation

alexkuzmik
Copy link
Collaborator

No description provided.

@alexkuzmik alexkuzmik self-assigned this Dec 4, 2023
@alexkuzmik alexkuzmik merged commit 2e18e78 into main Dec 4, 2023
15 checks passed
alexkuzmik added a commit that referenced this pull request Dec 4, 2023
* Add new configuration for tls verification, inject Session dependency into RestApiClient class

* Fix lint errors

* Refactor session object setup

* Bypass cache decorator in tests
alexkuzmik added a commit that referenced this pull request Dec 4, 2023
* Add env setup to openai tests workflow

* Remove old fixture usage

* Move env setup to tests step

* Add debug print for integration test

* Add -ss arg to pytest

* Add debug print for integration test

* Fix log call

* Add log

* Move env setup to 1 level above

* Move env setup to 1 level above

* Delete env setup from runner

* Move env setup to runner

* Revert last change

* Add secrets: inherit

* Remove debug logs

* Remove -ss arg in pytest command

* Reorganize request exception handling logic

* Remove odd comment

* [CM-9020] add config for disabling TLS verification (#107)

* Add new configuration for tls verification, inject Session dependency into RestApiClient class

* Fix lint errors

* Refactor session object setup

* Bypass cache decorator in tests

* Change log message in module loader
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

Successfully merging this pull request may close these issues.

None yet

1 participant