Skip to content

Only return valid stopwatches to the EventSource#19863

Merged
lunny merged 2 commits into
go-gitea:mainfrom
zeripath:fix-apiformat-stopwatch-errors
Jun 2, 2022
Merged

Only return valid stopwatches to the EventSource#19863
lunny merged 2 commits into
go-gitea:mainfrom
zeripath:fix-apiformat-stopwatch-errors

Conversation

@zeripath
Copy link
Copy Markdown
Contributor

@zeripath zeripath commented Jun 1, 2022

Looking through the logs of try.gitea.io I am seeing a number of reports
of being unable to APIformat stopwatches because the issueID is 0. These
are invalid StopWatches and they represent a db inconsistency.

This PR simply stops sending them to the eventsource.

Signed-off-by: Andrew Thornton art27@cantab.net

Looking through the logs of try.gitea.io I am seeing a number of reports
of being unable to APIformat stopwatches because the issueID is 0. These
are invalid StopWatches and they represent a db inconsistency.

This PR simply stops sending them to the eventsource.

Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath zeripath added this to the 1.17.0 milestone Jun 1, 2022
@zeripath zeripath changed the title Only return valid stopwatches Only return valid stopwatches to the EventSource Jun 1, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jun 2, 2022
@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 Jun 2, 2022
@lunny lunny merged commit 8aaba65 into go-gitea:main Jun 2, 2022
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jun 2, 2022
* giteaofficial/main:
  update documents (go-gitea#19868)
  Only return valid stopwatches (go-gitea#19863)
  [skip ci] Updated translations via Crowdin
  Add title attribute to dependencies in sidebar (go-gitea#19807)
@zeripath zeripath deleted the fix-apiformat-stopwatch-errors branch June 2, 2022 18:23
AbdulrhmnGhanem pushed a commit to kitspace/gitea that referenced this pull request Aug 24, 2022
Looking through the logs of try.gitea.io I am seeing a number of reports
of being unable to APIformat stopwatches because the issueID is 0. These
are invalid StopWatches and they represent a db inconsistency.

This PR simply stops sending them to the eventsource.

Signed-off-by: Andrew Thornton <art27@cantab.net>

Co-authored-by: techknowlogick <techknowlogick@gitea.io>
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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.

4 participants