Skip to content

Commit

Permalink
We now ignore invalid query params
Browse files Browse the repository at this point in the history
  • Loading branch information
Eric-Warehime committed Oct 28, 2022
1 parent 582471f commit f617d87
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion test/scripts/e2e_subs/rest-applications-endpoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -31,5 +31,5 @@ call_and_verify "App parameter parsing error 1." "/v2/applications/-2" 400 "Inva
call_and_verify "App parameter parsing error 2." "/v2/applications/not-a-number" 400 "Invalid format for parameter application-id"

# Good request, but invalid query parameters
call_and_verify "App invalid parameter" "/v2/applications/$APPID?this-should-fail=200" 400 'Unknown parameter detected: this-should-fail'
call_and_verify "App invalid parameter" "/v2/applications/$APPID?this-should-not-fail=200" 200 '"global-state-schema":{"num-byte-slice":0,"num-uint":2}'

2 changes: 1 addition & 1 deletion test/scripts/e2e_subs/rest-assets-endpoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -30,5 +30,5 @@ call_and_verify "Asset parameter parsing error 1." "/v2/assets/-2" 400 "Invalid
call_and_verify "Asset parameter parsing error 2." "/v2/assets/not-a-number" 400 "Invalid format for parameter asset-id"

# Good request, but invalid query parameters
call_and_verify "Asset invalid parameter" "/v2/assets/$ASSET_ID?this-should-fail=200" 400 'parameter detected: this-should-fail'
call_and_verify "Asset invalid parameter" "/v2/assets/$ASSET_ID?this-should-not-fail=200" 200 '","decimals":19,"default-frozen":false,"freeze":"'

0 comments on commit f617d87

Please sign in to comment.