Skip to content

[OPIK-515] implement opik client auth_check method#805

Merged
alexkuzmik merged 8 commits intomainfrom
OPIK-515-sdk-implement-opik-client-auth-check-method
Dec 4, 2024
Merged

[OPIK-515] implement opik client auth_check method#805
alexkuzmik merged 8 commits intomainfrom
OPIK-515-sdk-implement-opik-client-auth-check-method

Conversation

@alexkuzmik
Copy link
Copy Markdown
Collaborator

@alexkuzmik alexkuzmik commented Dec 3, 2024

Details

Adds auth_check() method to Opik. The method returns None if everything if the configured workspace is accessible for the configured API key, and raises an ApiError with the error message and status code from the backend if there is a problem with access.

Added e2e tests.
Also added a workaround for Fern bug with ignored httpx timeouts.

@alexkuzmik alexkuzmik requested a review from a team as a code owner December 3, 2024 17:07
andrescrz
andrescrz previously approved these changes Dec 3, 2024
Copy link
Copy Markdown
Member

@andrescrz andrescrz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@alexkuzmik alexkuzmik requested review from a team as code owners December 3, 2024 17:38
@alexkuzmik alexkuzmik marked this pull request as draft December 3, 2024 17:39
@alexkuzmik alexkuzmik marked this pull request as ready for review December 4, 2024 12:12
@alexkuzmik alexkuzmik merged commit d5e1ac9 into main Dec 4, 2024
@alexkuzmik alexkuzmik deleted the OPIK-515-sdk-implement-opik-client-auth-check-method branch December 4, 2024 12:39
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.

3 participants