Skip to content

Add Client.timeout property. - #1135

Merged
lovelydinosaur merged 1 commit into
encode:masterfrom
cdeler:make_base_client_auth_and_timeout_accesible_through_getter_and_setter
Aug 6, 2020
Merged

lovelydinosaur merged 1 commit into
encode:masterfrom
cdeler:make_base_client_auth_and_timeout_accesible_through_getter_and_setter

Conversation

@cdeler

@cdeler cdeler commented Aug 6, 2020

Copy link
Copy Markdown
Contributor

The PR is for #1066

@lovelydinosaur

Copy link
Copy Markdown
Member

Ah, thanks good stuff.
So I'd issue separate pull requests for auth and for timeouts here, because...

  • It think the Timeout PR is a clear-cut "yes please!"
  • It's not 100% clear to me if we want client.auth to return Optional[AuthTypes] or if it should instead always return the auth class instance. This one probably needs some explaining / discussion.

Perhaps have this PR but just tackling the timeouts, and we can think about auth seperately.

@cdeler
cdeler force-pushed the make_base_client_auth_and_timeout_accesible_through_getter_and_setter branch 2 times, most recently from 6465a29 to 258bc70 Compare August 6, 2020 09:57
@cdeler

cdeler commented Aug 6, 2020 •

Copy link
Copy Markdown
Contributor Author

It's not 100% clear to me if we want client.auth to return Optional[AuthTypes] or if it should instead always return the auth class instance. This one probably needs some explaining / discussion.

yes, I'll re-edit the PR

@cdeler
cdeler force-pushed the make_base_client_auth_and_timeout_accesible_through_getter_and_setter branch from 258bc70 to 5191dc5 Compare August 6, 2020 10:05
@cdeler
cdeler force-pushed the make_base_client_auth_and_timeout_accesible_through_getter_and_setter branch from 5191dc5 to de8652e Compare August 6, 2020 10:31
@lovelydinosaur lovelydinosaur changed the title #1066 make BaseClient's auth and proxy accessible using getters and setters Add Client.timeout property. Aug 6, 2020

@lovelydinosaur lovelydinosaur left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fantastic, yup - lovely little change.

@lovelydinosaur
lovelydinosaur merged commit 7123b0f into encode:master Aug 6, 2020
@cdeler
cdeler deleted the make_base_client_auth_and_timeout_accesible_through_getter_and_setter branch September 9, 2020 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants