Merge pull request #1 from ForAllSecure/sarif-report

Fix mapi SARIF report upload for codeQL

Author: xansec

.github/workflows/mapi.yml

@@ -49,6 +49,13 @@ jobs:
         name: mapi-report
         path: mapi.html
 
+    # Archive SARIF report
+    - name: Archive Mayhem for API SARIF report
+      uses: actions/upload-artifact@v2
+      with:
+        name: sarif-report
+        path: mapi.sarif     
+
     # Upload SARIF file (only available on public repos or github enterprise)
     - name: Upload SARIF file
       uses: github/codeql-action/upload-sarif@v1

server/main.go

@@ -129,7 +129,9 @@ func (s *server) DeleteUser(ctx context.Context, in *pb.DeleteUserRequest) (*pb.
 		return nil, status.Error(codes.NotFound, fmt.Sprintf("Could not find user with ID - %d", in.GetId()))
 	}
 
-	return &pb.DeleteUserResult{Count: count}, nil
+	return nil, status.Error(codes.Internal, "Failed to delete user anyway")
+
+	// return &pb.DeleteUserResult{Count: count}, nil
 }
 
 //