[MI-2877]: Review fixes

mattermost-community · Mar 13, 2023 · f53bbd3 · f53bbd3
1 parent 2c7bf28
commit f53bbd3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/server/constants/constants.go b/server/constants/constants.go
@@ -171,7 +171,7 @@ const (
 
 	DialogFieldNameComment = "comment"
 
-	MaxBytesSizeForReadingClientBody = 1000000
+	MaxBytesSizeForReadingResponseBody = 1000000
 )
 
 var (

diff --git a/server/plugin/client.go b/server/plugin/client.go
@@ -497,9 +497,9 @@ func (c *client) makeHTTPRequest(req *http.Request, contentType string, out inte
 	}
 	defer resp.Body.Close()
 
-	// Limit client bodies to 1 MB or 1000 KB
+	// Limit reading the response bodies to 1 MB or 1000 KB
 	// This is ideal for the responses returned by Azure DevOps APIs used here
-	resp.Body = http.MaxBytesReader(nil, resp.Body, constants.MaxBytesSizeForReadingClientBody)
+	resp.Body = http.MaxBytesReader(nil, resp.Body, constants.MaxBytesSizeForReadingResponseBody)
 
 	responseData, err = io.ReadAll(resp.Body)
 	if err != nil {
-Original file line number
+Diff line change
@@ Expand Up / @@ -171,7 +171,7 @@ const ( @@
     	DialogFieldNameComment = "comment"
-    	MaxBytesSizeForReadingClientBody = 1000000
+    	MaxBytesSizeForReadingResponseBody = 1000000
     )
     var (
@@ Expand Down @@