We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I'm using grpc-gateway patch feature so it generates following openapi spec
"/v1/public/auth/user/{user.id}": { "put": { "summary": "Update user", "operationId": "AuthService_UpdateUser", "responses": { ... }, "parameters": [ { "name": "user.id", "in": "path", "required": true, "type": "string", "format": "uint64" }, ... ] }, "patch": { "summary": "Update user", "operationId": "AuthService_UpdateUser2", "responses": { ... }, "parameters": [ { "name": "user.id", "in": "path", "required": true, "type": "string", "format": "uint64" }, ... ], } },
As we can see there is {user.id} parameter in the path, but js client does not replace it with correct variable. According to this lines Javascript/ApiClient.mustache#L147 Javascript/es6/ApiClient.mustache#L137 it replaces only {user_id} like params
{user.id}
{user_id}
Changing regexp to /\{([\w-\.]+)\}/g will solve the problem
/\{([\w-\.]+)\}/g
The text was updated successfully, but these errors were encountered:
Changing regexp to /{([\w-.]+)}/g will solve the problem
can you please file a PR with the suggested fix so that we can review it more easily?
Sorry, something went wrong.
Hi, please look PR ^^
Successfully merging a pull request may close this issue.
Is your feature request related to a problem? Please describe.
I'm using grpc-gateway patch feature so it generates following openapi spec
As we can see there is
{user.id}
parameter in the path, but js client does not replace it with correct variable.According to this lines
Javascript/ApiClient.mustache#L147
Javascript/es6/ApiClient.mustache#L137
it replaces only
{user_id}
like paramsDescribe the solution you'd like
Changing regexp to
/\{([\w-\.]+)\}/g
will solve the problemDescribe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered: