Skip to content

Commit

Permalink
fix: allow cloudsearchdomain POST search compatibility in unit tests (#…
Browse files Browse the repository at this point in the history
…4625)

* fix: allow cloudsearchdomain POST search compatibility in unit tests

* fix: exit convertGetToPost if request is already POST
  • Loading branch information
kuhe committed Apr 2, 2024
1 parent 074aef9 commit d7b9e64
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 25 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
.idea
configuration
node_modules
npm-debug.log
Expand Down
1 change: 1 addition & 0 deletions .npmignore
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
.npmignore
.tesselinclude
.vscode/
.idea
apis/*.normal.json
appveyor.yml
bower.json
Expand Down
3 changes: 3 additions & 0 deletions lib/services/cloudsearchdomain.js
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,9 @@ AWS.util.update(AWS.CloudSearchDomain.prototype, {
*/
convertGetToPost: function(request) {
var httpRequest = request.httpRequest;
if (httpRequest.method === 'POST') {
return;
}
// convert queries to POST to avoid length restrictions
var path = httpRequest.path.split('?');
httpRequest.method = 'POST';
Expand Down
13 changes: 1 addition & 12 deletions test/services/cloudsearchdomain.spec.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 0 additions & 13 deletions test/signers/v4.spec.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit d7b9e64

Please sign in to comment.