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.

Sign up for GitHub

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

Jump to bottom

Sync branches to DB immediately when handle git hook calling #29493

Merged
merged 12 commits into from
Mar 6, 2024
Merged

Sync branches to DB immediately when handle git hook calling #29493

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
6bef773
feat: sync to db in hook
wolfogre Feb 29, 2024
3d68828
Update routers/private/hook_post_receive.go
wolfogre Mar 1, 2024
4fb19f7
fix: ignore deleted
wolfogre Mar 1, 2024
62d3cfa
test: debug log
wolfogre Mar 1, 2024
a844be5
Revert "test: debug log"
wolfogre Mar 1, 2024
75950d0
feat: add deleted branches
wolfogre Mar 1, 2024
49f224a
fix: improve performance of SyncBranchesToDB
wolfogre Mar 1, 2024
0cfa28f
chore: lint
wolfogre Mar 1, 2024
d1bb3e0
fix: check if never synced
wolfogre Mar 4, 2024
7d7bdeb
test: TestGitPush
wolfogre Mar 5, 2024
b204dbb
chore: lint
wolfogre Mar 5, 2024
0b2033c
Merge branch 'main' into bugfix/sync_branch
GiteaBot Mar 6, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update routers/private/hook_post_receive.go 
Co-authored-by: Lunny Xiao <[email protected]>
  • Loading branch information
@wolfogre @lunny
wolfogre and lunny committed Mar 1, 2024
commit 3d6882822f73c15191c6c7b970da272e0281c36b
13 changes: 8 additions & 5 deletions routers/private/hook_post_receive.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,12 +116,15 @@ func HookPostReceive(ctx *gitea_context.PrivateContext) {
return
}
}
commit, err := gitRepo.GetCommit(update.NewCommitID)
if err != nil {
ctx.JSON(http.StatusInternalServerError, private.HookPostReceiveResult{
var commit *git.Commit
if !opts.IsDelRef() {
commit, err := gitRepo.GetCommit(update.NewCommitID)
if err != nil {
ctx.JSON(http.StatusInternalServerError, private.HookPostReceiveResult{
Err: fmt.Sprintf("Failed to get commit %s in repository: %s/%s Error: %v", update.NewCommitID, ownerName, repoName, err),
})
return
})
return
}
}
wolfogre marked this conversation as resolved.
Show resolved Hide resolved
if err := repo_service.SyncBranchToDB(ctx, repo.ID, update.PusherID, update.RefFullName.BranchName(), commit); err != nil {
ctx.JSON(http.StatusInternalServerError, private.HookPostReceiveResult{
Expand Down
Loading