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

[Bug] Incorrect Git command default timeout #10575

Closed
2 of 7 tasks
camo-f opened this issue Mar 2, 2020 · 1 comment · Fixed by #10581
Closed
2 of 7 tasks

[Bug] Incorrect Git command default timeout #10575

camo-f opened this issue Mar 2, 2020 · 1 comment · Fixed by #10581
Labels
type/bug type/docs This PR mainly updates/creates documentation

Comments

@camo-f
Copy link

camo-f commented Mar 2, 2020

Hi there,

  • Gitea version (or commit ref): 1.11.1
  • Git version: 2.24.1
  • Operating system: Docker
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant

Description

In the documentation, it is specified that git.timeout.default is set to 360 seconds. See https://docs.gitea.io/en-us/config-cheat-sheet/#git-timeout-settings-git-timeout
This is also the value documented here : https://github.com/go-gitea/gitea/blob/master/custom/conf/app.ini.sample#L877

This is not the case when starting the application without this setting, where the default is 60 :
The value is defined here : https://github.com/go-gitea/gitea/blob/master/modules/git/command.go#L25
Here default timeout is set with the previous value : https://github.com/go-gitea/gitea/blob/master/modules/setting/git.go#L53

This is quite misleading (in my case my update hook was crashing after 60 seconds without any apparent reason). Either the code or the documentation should be updated with the correct value (60 or 360).

A bit off-topic, but is there a reason why it is not possible to set every configuration value with Docker environment variables ?

Thanks in advance !

@lunny
Copy link
Member

lunny commented Mar 2, 2020

360 is more reasonable currently.

@guillep2k guillep2k added type/bug type/docs This PR mainly updates/creates documentation labels Mar 3, 2020
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type/bug type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants