AWS codeguru documentation change
Summary
Added input validation patterns (\S+) to commit SHA fields and changed required status between MergeBaseCommit/SourceCommit fields
Security assessment
Added regex pattern validation improves input sanitization but there's no explicit mention of addressing a specific vulnerability. Changing required fields relates to API contract rather than direct security fixes.
Diff
diff --git a/codeguru/latest/reviewer-api/API_CommitDiffSourceCodeType.md b/codeguru/latest/reviewer-api/API_CommitDiffSourceCodeType.md index 6715ea253..bc67d052c 100644 --- a//codeguru/latest/reviewer-api/API_CommitDiffSourceCodeType.md +++ b//codeguru/latest/reviewer-api/API_CommitDiffSourceCodeType.md @@ -22 +22 @@ Length Constraints: Minimum length of 6. Maximum length of 64. -Required: No +Pattern: `\S+` @@ -24 +24 @@ Required: No -**MergeBaseCommit** +Required: Yes @@ -25,0 +26 @@ Required: No +**SourceCommit** @@ -27 +28,2 @@ Required: No -The SHA of the merge base of a commit. + +The SHA of the source commit used to generate a commit diff. This field is required for a pull request code review. @@ -33 +35 @@ Length Constraints: Minimum length of 6. Maximum length of 64. -Required: No +Pattern: `\S+` @@ -35 +37 @@ Required: No -**SourceCommit** +Required: Yes @@ -36,0 +39 @@ Required: No +**MergeBaseCommit** @@ -38 +41,2 @@ Required: No -The SHA of the source commit used to generate a commit diff. This field is required for a pull request code review. + +The SHA of the merge base of a commit. @@ -43,0 +48,2 @@ Length Constraints: Minimum length of 6. Maximum length of 64. +Pattern: `\S+` +