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>tags/v1.16.0-rc1
@@ -283,7 +283,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 { |