Only push to DockerHub when commit on master branch (#4)

* Only push to Dockerhub on master

* Supress user nullable warning

* Use on pr condition instead

.github/workflows/docker-build.yml

@@ -1,18 +1,25 @@
-name: docker-build
-on: [push]
+name: Docker Build Image
+
+on:
+  pull_request:
+    branches:
+      - master
+
 jobs:
   build:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
+
       - name: Docker Login
         uses: docker/[email protected]
         with:
           username: ${{ secrets.DOCKERHUB_USERNAME }}
           password: ${{ secrets.DOCKERHUB_TOKEN }}
-      - name: Build and Push
+
+      - name: Build Image
         uses: docker/[email protected]
         with:
           file: dockerfile
           tags: ${{ secrets.DOCKERHUB_USERNAME }}/thuinfo-web:latest
-          push: true
+          push: false

.github/workflows/docker-push.yml

@@ -0,0 +1,25 @@
+name: Docker Build and Push Image to DockerHub
+
+on:
+  push:
+    branches:
+      - master
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Docker Login
+        uses: docker/[email protected]
+        with:
+          username: ${{ secrets.DOCKERHUB_USERNAME }}
+          password: ${{ secrets.DOCKERHUB_TOKEN }}
+
+      - name: Build Image and Push to DockerHub
+        uses: docker/[email protected]
+        with:
+          file: dockerfile
+          tags: ${{ secrets.DOCKERHUB_USERNAME }}/thuinfo-web:latest
+          push: true

ThuInfoWeb/Controllers/HomeController.cs

@@ -145,12 +145,12 @@ public async Task<IActionResult> CreateAnnounce(AnnounceViewModel vm)
             if (vm.Title is null || vm.Content is null) return BadRequest("标题或内容为空");
             vm.VisibleNotAfter ??= "9.9.9";
             vm.VisibleExact ??= "";
-            var user = HttpContext.User.Identity?.Name;
+            var user = HttpContext.User.Identity!.Name!;
             var a = new Announce
             {
                 Title = vm.Title,
                 Content = vm.Content,
-                Author = user ?? "",
+                Author = user,
                 CreatedTime = DateTime.Now,
                 IsActive = vm.IsActive,
                 VisibleNotAfter = vm.VisibleNotAfter,