serverless-openapi-documentation icon indicating copy to clipboard operation
serverless-openapi-documentation copied to clipboard

Examples don't work, possible bug

Open piersmacdonald opened this issue 6 years ago • 5 comments

The serverless.docs.yml and serverless.yml duplicate the same documentation and it's not clear which was used to generate the openapi.yml in the directory.

When I run the sample my createUser endpoint fails to create a requestBody , I can't replicate the openapi.yml in the test/project directory. If I run the serverless.yml in the sample I get this in my output

    post:
      operationId: createUser
      summary: Create User
      description: Creates a user and then sends a generated password email
      responses:
        '201':
          description: A user object along with generated API Keys
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/PutDocumentResponse'
        '500':
          description: An error message when creating a new user
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/ErrorResponse'```

Without this I'm not sure if there's a reason I can't add a request body to an endpoint

piersmacdonald avatar Jul 11 '18 22:07 piersmacdonald

+1

pig800509 avatar Jul 18 '18 05:07 pig800509

There is a bug since #16 which ignores request body and path parameters which is solved by PR #24.

David-Sellen avatar Jul 23 '18 07:07 David-Sellen

@David-Sellen I don't see it is solved by now, which version do you use? I run serverless openapi generate -o swagger.json -f json and this is path of swagger.json

"paths": {
    "/create": {
      "post": {
        "operationId": "createUser",
        "summary": "Create User",
        "description": "Creates a user and then sends a generated password email",
        "responses": {
          "201": {
            "description": "A user object along with generated API Keys",
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/PutDocumentResponse"
                }
              }
            }
          },
          "500": {
            "description": "An error message when creating a new user",
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/ErrorResponse"
                }
              }
            }
          }
        }
      }
    }
  }

pig800509 avatar Jul 24 '18 09:07 pig800509

@pig800509 that is because the fix has not been merged into this repo

David-Sellen avatar Jul 24 '18 09:07 David-Sellen

@David-Sellen I see, thank you so much.

pig800509 avatar Jul 24 '18 14:07 pig800509