fixbug

routers/web/shared/actions/runners.go

@@ -156,6 +156,7 @@ func RunnerRegistARunner(ctx *context.Context, ownerID, repoID int64, redirectTo
 		ctx.Redirect(redirectTo)
 		return
 	}
+
 	ctx.Flash.Success(ctx.Tr("actions.runners.regist_runner_success"))
 	ctx.Redirect(redirectTo)
 }
@@ -164,6 +165,12 @@ func RunnerRegistARunner(ctx *context.Context, ownerID, repoID int64, redirectTo
 func RunnerDeletePost(ctx *context.Context, runnerID int64,
 	successRedirectTo, failedRedirectTo string,
 ) {
+
+	runner, err := actions_model.GetRunnerByID(ctx, runnerID)
+	if err != nil {
+		log.Warn("DeleteRunnerPost GetRunnerID failed")
+	}
+
 	if err := actions_model.DeleteRunner(ctx, runnerID); err != nil {
 		log.Warn("DeleteRunnerPost.UpdateRunner failed: %v, url: %s", err, ctx.Req.URL)
 		ctx.Flash.Warning(ctx.Tr("actions.runners.delete_runner_failed"))
@@ -171,10 +178,7 @@ func RunnerDeletePost(ctx *context.Context, runnerID int64,
 		ctx.JSONRedirect(failedRedirectTo)
 		return
 	}
-	runner, err := actions_model.GetRunnerByID(ctx, runnerID)
-	if err != nil {
-		log.Warn("DeleteRunnerPost.DeleteRunnerContainer failed: %v, url: %s", err, ctx.Req.URL)
-	}
+
 	runnerName := runner.Name
 
 	err = runner_services.DeleteRunnerByName(ctx, runnerName)
@@ -185,6 +189,6 @@ func RunnerDeletePost(ctx *context.Context, runnerID int64,
 	log.Info("DeleteRunnerPost success: %s", ctx.Req.URL)
 
 	ctx.Flash.Success(ctx.Tr("actions.runners.delete_runner_success"))
-
+	log.Info("DeleteRunnerPost success: %s", successRedirectTo)
 	ctx.JSONRedirect(successRedirectTo)
 }