Ensure default gpg settings not nil and found commits have reference to repo (#8604)
* Ensure defaultGPGSettings not nil * Ensure that coerced commits gain a reference to the repo * Add warning if trying to get defaultgpgsetting on an unattached commit
This commit is contained in:
		
							parent
							
								
									c8f3146cd5
								
							
						
					
					
						commit
						28f60bb5cb
					
				
					 2 changed files with 3 additions and 0 deletions
				
			
		|  | @ -682,6 +682,8 @@ func ParseCommitWithSignature(c *git.Commit) *CommitVerification { | |||
| 	defaultGPGSettings, err := c.GetRepositoryDefaultPublicGPGKey(false) | ||||
| 	if err != nil { | ||||
| 		log.Error("Error getting default public gpg key: %v", err) | ||||
| 	} else if defaultGPGSettings == nil { | ||||
| 		log.Warn("Unable to get defaultGPGSettings for unattached commit: %s", c.ID.String()) | ||||
| 	} else if defaultGPGSettings.Sign { | ||||
| 		if commitVerification := verifyWithGPGSettings(defaultGPGSettings, sig, c.Signature.Payload, committer, keyID); commitVerification != nil { | ||||
| 			if commitVerification.Reason == BadSignature { | ||||
|  |  | |||
|  | @ -72,6 +72,7 @@ func (tes Entries) GetCommitsInfo(commit *Commit, treePath string, cache LastCom | |||
| 		treeCommit = commit | ||||
| 	} else if rev, ok := revs[""]; ok { | ||||
| 		treeCommit = convertCommit(rev) | ||||
| 		treeCommit.repo = commit.repo | ||||
| 	} | ||||
| 	return commitsInfo, treeCommit, nil | ||||
| } | ||||
|  |  | |||
		Loading…
	
		Reference in a new issue