Check if telemetry is disable, and only submit the true up profile if it is disabled.

server/channels/api4/license.go

@@ -351,12 +351,15 @@ func requestTrueUpReview(c *Context, w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	// True-up is only enabled when telemetry is disabled. When telemetry is enabled, we already have all the data necessary
+	// True-up is only enabled when telemetry is disabled.
-	// for true-up reviews to be completed.
+	// When telemetry is enabled, we already have all the data necessary for true-up reviews to be completed.
-	err = c.App.Cloud().SubmitTrueUpReview(c.AppContext.Session().UserId, profileMap)
+	telemetryEnabled := c.App.Config().LogSettings.EnableDiagnostics
-	if err != nil {
+	if telemetryEnabled != nil && !*telemetryEnabled {
-		c.Err = model.NewAppError("requestTrueUpReview", "api.license.true_up_review.failed_to_submit", nil, err.Error(), http.StatusInternalServerError)
+		err = c.App.Cloud().SubmitTrueUpReview(c.AppContext.Session().UserId, profileMap)
-		return
+		if err != nil {
+			c.Err = model.NewAppError("requestTrueUpReview", "api.license.true_up_review.failed_to_submit", nil, err.Error(), http.StatusInternalServerError)
+			return
+		}
 	}
 
 	// Update the review status to reflect the completion.