Skip to content
This repository has been archived by the owner on Nov 9, 2023. It is now read-only.

Discuss a reasonable default timeout #4

Closed
rauchg opened this issue Nov 2, 2017 · 2 comments
Closed

Discuss a reasonable default timeout #4

rauchg opened this issue Nov 2, 2017 · 2 comments
Labels
enhancement question

Comments

@rauchg
Copy link
Member

rauchg commented Nov 2, 2017

I think the current one is too high (socket timeout I believe). I think maybe 5s would be a good timeout?
@MunifTanjim MunifTanjim mentioned this issue Oct 14, 2018
Closed
Ethan-Arrowood pushed a commit that referenced this issue Feb 10, 2022
@rauchg
Fix port (#4)
967ff4a 
* fix hostname setting in the url

* add deps

* add test

* add mandatory linting / prettifying

* add lint config

* fix linting
@Allanflo88
Copy link

I think 5 seconds is fine as well, but what about 3 seconds based on google statistics of user leaving websites that take longer to respond?

@Ethan-Arrowood
Copy link
Collaborator

Thank you for the report. We are archiving this project; you can read more details here: #83

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rauchg @Ethan-Arrowood @Allanflo88