bug(repo sync): ignore inactive repos

[KellyMerrick]

This closes go-vela/community#852.

Add bool to track if webhook has been deleted from SCM:

server/scm/github/repo.go

Lines 263 to 274 in f89e929

	// track if webhook cannot be captured from GitHub (no longer found),
	// indicating the webhook has been manually deleted from GitHub
	isWebhookDel := false
	if resp.StatusCode == http.StatusNotFound {
	isWebhookDel = true
	}
	if err != nil {
	return isWebhookDel, err
	}
	return isWebhookDel, nil

And then update status to inactive in the database:

server/api/scm/sync_org.go

Lines 140 to 164 in f89e929

	// update webhook
	isWebhookDel, err := scm.FromContext(c).Update(u, repo, lastHook.GetWebhookID())
	if err != nil {
	retErr := fmt.Errorf("unable to update repo webhook for %s: %w", repo.GetFullName(), err)
	util.HandleError(c, http.StatusInternalServerError, retErr)
	// if webhook has been manually deleted from GitHub,
	// set to inactive in database
	if isWebhookDel {
	repo.SetActive(false)
	_, err := database.FromContext(c).UpdateRepo(ctx, repo)
	if err != nil {
	retErr := fmt.Errorf("unable to update repo for org %s: %w", o, err)
	util.HandleError(c, http.StatusInternalServerError, retErr)
	return
	}
	}
	return
	}

[codecov]

Codecov Report

Merging #953 (08a7db6) into main (0464cb5) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #953      +/-   ##
==========================================
+ Coverage   70.95%   70.97%   +0.02%     
==========================================
  Files         312      312              
  Lines       12923    12922       -1     
==========================================
+ Hits         9170     9172       +2     
+ Misses       3287     3285       -2     
+ Partials      466      465       -1     

Files Changed	Coverage Δ
scm/github/repo.go	76.99% <100.00%> (+0.64%)	⬆️

📢 Have feedback on the report? Share it here.