SDK generation not handling URL path variables

[zjost]

When trying to recreate the example in the docs, the generated javascript GET methods like apigClient.helloNameGet({name: 'jimmy'}) that are supposed to use parameters to construct the path don't seem to work. When using chrome inspect it looks like it's not adding "jimmy" to the URL.

When constructing an example like @app.route('/{name}/hello') then the javascript makes an api call to <BASE>/api//hello.

[zjost]

Note: If I modify apigClient.js to add the variable name into the key list for object parsing, it seems to work. For instance, in apigClient.helloNameGet, I have to modify var helloNameGetRequest value for the path key to go from:

path: pathComponent + uritemplate('/hello/{name}').expand(apiGateway.core.utils.parseParametersToObject(params, [])),

to:

path: pathComponent + uritemplate('/hello/{name}').expand(apiGateway.core.utils.parseParametersToObject(params, ['name'])),

[jamesls]

Thanks, this is a confirmed bug. I believe this regressed when we switched over to generating a swagger file internally. I think the parameters field we generate isn't at the right level of nesting. It also appears that API Gateway ignores fields it doesn't understand:

What we generate

          "paths": {
            "/hello/{name}": {
              "get": {
                "responses": {
                  "200": {
                    "description": "200 response",
                    "schema": {
                      "$ref": "#/definitions/Empty"
                    }
                  }
                },
                "x-amazon-apigateway-integration": {
                  "contentHandling": "CONVERT_TO_TEXT",
                  "responses": {
                    "default": {
                      "statusCode": "200"
                    }
                  },
                  "parameters": [
                    {
                      "required": true,
                      "type": "string",
                      "name": "name",
                      "in": "path"
                    }
                  ],

What we actually need to generate

    "/hello/{name}": {
      "get": {
        "produces": [
          "application/json"
        ],
        "parameters": [
          {
            "name": "name",
            "in": "path",
            "required": true,
            "type": "string"
          }
        ],
        "responses": {
          "200": {
            "description": "200 response",
            "schema": {
              "$ref": "#/definitions/Empty"
            }
          }
        },
        "x-amazon-apigateway-integration": {
          "responses": {
            "default": {
              "statusCode": "200"
            }
          },

Note the parameters should be moved up a level.

Thanks for reporting we'll get this fixed.