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

Fixes the 30 seconds timeout when cloning over HTTP #1243

Merged
merged 2 commits into from Jun 5, 2017

Conversation

sebastianopilla
Copy link

Adds the

server.httpTimeout

setting to allow cloning big repositories over HTTP. This change fixes the java.util.concurrent.TimeoutException issue described in https://groups.google.com/d/topic/gitblit/UvDC48NpmF4/discussion .

server.httpTimeout

setting to allow cloning big repositories over HTTP. This change fixes the java.util.concurrent.TimeoutException issue described in https://groups.google.com/d/topic/gitblit/UvDC48NpmF4/discussion .
Copy link
Collaborator

@gitblit gitblit left a comment

Choose a reason for hiding this comment

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

Looks good. Would you change the name a bit?

#
# SINCE 1.9.0
# RESTART REQUIRED
server.httpTimeout = 30000
Copy link
Collaborator

Choose a reason for hiding this comment

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

How about server.httpIdleTimeout

@@ -2134,6 +2134,13 @@ server.requireClientCertificates = false
# RESTART REQUIRED
server.shutdownPort = 8081

# Timeout (in milliseconds) for http and https requests
Copy link
Collaborator

Choose a reason for hiding this comment

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

How about Http idle timeout (in milliseconds)...

@sebastianopilla
Copy link
Author

I've made the changes, I can't figure out if there's anything else that I need to do to have them applied to the pull request however. Let me know if I need to drop this pull request and create a new one.

@gitblit gitblit merged commit a13f078 into gitblit-org:master Jun 5, 2017
@gitblit
Copy link
Collaborator

gitblit commented Jun 5, 2017

Thanks!

@flaix flaix added this to the 1.9.0 milestone Nov 16, 2019
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

3 participants