Skip to content

Conversation

@6543
Copy link
Member

@6543 6543 commented Jul 26, 2021

while working on #16551 I found some session bugs at models

@6543 6543 added this to the 1.16.0 milestone Jul 26, 2021
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 26, 2021
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jul 26, 2021
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jul 26, 2021
@lafriks lafriks merged commit 7c7771e into go-gitea:main Jul 26, 2021
@lafriks lafriks deleted the fix-session-bugs branch July 26, 2021 20:46
6543 added a commit to 6543-forks/gitea that referenced this pull request Jul 26, 2021
* fix deadlog bug

* Fix models/issue_stopwatch.go

* Update models/issue_stopwatch.go

Co-authored-by: zeripath <[email protected]>
@6543 6543 mentioned this pull request Jul 26, 2021
@6543 6543 added the backport/done All backports for this PR have been created label Jul 26, 2021
@6543
Copy link
Member Author

6543 commented Jul 26, 2021

-> #16553

@lunny
Copy link
Member

lunny commented Jul 27, 2021

The bug is that no database transaction when create stop watch or cancel so that it may be non-consistent.

lunny pushed a commit that referenced this pull request Jul 27, 2021
* Fix session bugs (#16552)

* fix deadlog bug

* Fix models/issue_stopwatch.go

* Update models/issue_stopwatch.go

Co-authored-by: zeripath <[email protected]>

* fix getLatestCommitStatus

Co-authored-by: zeripath <[email protected]>
AbdulrhmnGhanem pushed a commit to kitspace/gitea that referenced this pull request Aug 10, 2021
* fix deadlog bug

* Fix models/issue_stopwatch.go

* Update models/issue_stopwatch.go

Co-authored-by: zeripath <[email protected]>
@go-gitea go-gitea locked and limited conversation to collaborators Oct 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants