diff --git a/IRaCIS.Core.API/appsettings.Test_Study.json b/IRaCIS.Core.API/appsettings.Test_Study.json
index 095d3c718..cf0f19818 100644
--- a/IRaCIS.Core.API/appsettings.Test_Study.json
+++ b/IRaCIS.Core.API/appsettings.Test_Study.json
@@ -7,7 +7,8 @@
     }
   },
   "AliyunOSS": {
-    "endpoint": "http://oss-cn-shanghai.aliyuncs.com",
+    "endpoint": "https://oss-cn-shanghai.aliyuncs.com",
+    "region": "oss-cn-shanghai",
     "accessKeyId": "mpXG7Nu6zTpsDrI1",
     "accessKeySecret": "yNINcEb099SkNfF6vYKaoP8TZNI3xZ",
     "bucketName": "zyypacs"
diff --git a/IRaCIS.Core.API/appsettings.json b/IRaCIS.Core.API/appsettings.json
index a2828b8c7..b363c8ad5 100644
--- a/IRaCIS.Core.API/appsettings.json
+++ b/IRaCIS.Core.API/appsettings.json
@@ -5,12 +5,6 @@
     "Audience": "ZhiZhun",
     "TokenExpireDays": "7"
   },
-  "AliyunOSS": {
-    "endpoint": "http://oss-cn-shanghai.aliyuncs.com",
-    "accessKeyId": "mpXG7Nu6zTpsDrI1",
-    "accessKeySecret": "yNINcEb099SkNfF6vYKaoP8TZNI3xZ",
-    "bucketName": "zyypacs"
-  },
   "IpRateLimiting": {
     "EnableEndpointRateLimiting": true,
     "StackBlockedRequests": false,
diff --git a/IRaCIS.Core.Application/Triggers/VisitTaskIAfterSignTrigger.cs b/IRaCIS.Core.Application/Triggers/VisitTaskIAfterSignTrigger.cs
index ab1171212..4cac144c0 100644
--- a/IRaCIS.Core.Application/Triggers/VisitTaskIAfterSignTrigger.cs
+++ b/IRaCIS.Core.Application/Triggers/VisitTaskIAfterSignTrigger.cs
@@ -49,7 +49,7 @@ namespace IRaCIS.Core.Application.Triggers
         {
             var visitTask = context.Entity;
 
-            if (_userInfo.UserTypeEnumInt == (int)UserTypeEnum.PI)
+            if (context.ChangeType == ChangeType.Modified && visitTask.ReadingTaskState==ReadingTaskState.HaveSigned && visitTask.ReadingTaskState != context.UnmodifiedEntity.ReadingTaskState    &&        _userInfo.UserTypeEnumInt == (int)UserTypeEnum.PI)
             {
                 visitTask.PIAuditState = PIAuditState.PIAgree;
             }