PR(DROP): Patch The SourceHub Bug Temporarily

Must not merge with this commit in the PR.
shahzadlone · Mar 5, 2024 · 8b3dcf1 · 8b3dcf1
1 parent 7c8a5d7
commit 8b3dcf1
Showing 1 changed file with 13 additions and 10 deletions.
diff --git a/acp/acp_local.go b/acp/acp_local.go
@@ -329,16 +329,19 @@ func (l *ACPLocalEmbedded) CheckDocAccess(
 		&checkDoc,
 	)
 	if err != nil {
-		log.ErrorE(
-			ctx,
-			"failed to check/verify doc access with local acp module",
-			err,
-			logging.NewKV("PolicyID", policyID),
-			logging.NewKV("ActorID", actorID),
-			logging.NewKV("Resource", resource),
-			logging.NewKV("DocID", docID),
-		)
-		return false, err
+		//log.ErrorE(
+		//	ctx,
+		//	"failed to check/verify doc access with local acp module",
+		//	err,
+		//	logging.NewKV("PolicyID", policyID),
+		//	logging.NewKV("ActorID", actorID),
+		//	logging.NewKV("Resource", resource),
+		//	logging.NewKV("DocID", docID),
+		//)
+		// return false, err
+		// TODO-MUST: This change must be dropped once SourceHub fixes the bug.
+		// For now patching it to pass the tests that use wrong ids to test those cases.
+		return false, nil
 	}
 
 	if checkDocResponse.Valid {