Backport #17434 Unfortunately there was a regression in #17373 which missed that the user is not for deploy keys. This leads to a panic when pushing with deploy keys. Fix #17412 Signed-off-by: Andrew Thornton <art27@cantab.net>
This commit is contained in:
		
							parent
							
								
									1fbdf96c34
								
							
						
					
					
						commit
						dd1ba34ee5
					
				
					 1 changed files with 1 additions and 1 deletions
				
			
		|  | @ -282,7 +282,7 @@ func ServCommand(ctx *context.PrivateContext) { | |||
| 		(mode > models.AccessModeRead || | ||||
| 			repo.IsPrivate || | ||||
| 			owner.Visibility.IsPrivate() || | ||||
| 			user.IsRestricted || | ||||
| 			(user != nil && user.IsRestricted) || // user will be nil if the key is a deploykey
 | ||||
| 			setting.Service.RequireSignInView) { | ||||
| 		if key.Type == models.KeyTypeDeploy { | ||||
| 			if deployKey.Mode < mode { | ||||
|  |  | |||
		Loading…
	
		Reference in a new issue