Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

provider/aws: AWS API Gateway request parameters json #6501

Merged

Conversation

gthole
Copy link
Contributor

@gthole gthole commented May 5, 2016

Add the ability to specify request parameters on API Gateway Method and Integration resources, which achieves parity with MethodResponse and IntegrationResponse resources and allows incoming headers and query string parameters to be passed to the backend controller.

This change is a straight extension of the work done in #6344.

@gthole gthole changed the title AWS API Gateway request parameters json provider/aws: AWS API Gateway request parameters json May 5, 2016
@francknouama
Copy link

👍 🎆 I neeeed it

@mgribouski
Copy link

👍

@@ -134,6 +140,14 @@ func resourceAwsApiGatewayMethodUpdate(d *schema.ResourceData, meta interface{})
operations = append(operations, expandApiGatewayRequestResponseModelOperations(d, "request_models", "requestModels")...)
}

if d.HasChange("request_parameters_in_json") {
ops, err := expandApiGatewayMethodResponseParametersJSONOperations(d, "request_parameters_in_json", "requestParameters")
Copy link
Member

@radeksimko radeksimko May 5, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should rename the function to make it more accurate:
expandApiGatewayMethodResponseParametersJSONOperations => expandApiGatewayMethodParametersJSONOperations

(i.e. it is now being used for both response and request parameters so it should be called accordingly)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Good call.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and also there are two error messages inside the body of that function that should be changed accordingly too, so they're reflecting the reality.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done 😃

@radeksimko
Copy link
Member

radeksimko commented May 5, 2016

I'm currently running acceptance tests for API Gateway, but the code is looking good. ⏳

@radeksimko radeksimko merged commit ebfc701 into hashicorp:master May 5, 2016
cristicalin pushed a commit to cristicalin/terraform that referenced this pull request May 24, 2016
* Update docs with new parameters

* Add request parameters as JSON

* Update function name and error statements
@ghost
Copy link

ghost commented Apr 26, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants