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

Wrong endpoint paths in swagger files #407

Open
ma1112 opened this issue Mar 11, 2025 · 0 comments · May be fixed by #408
Open

Wrong endpoint paths in swagger files #407

ma1112 opened this issue Mar 11, 2025 · 0 comments · May be fixed by #408
Labels

Comments

@ma1112
Copy link

ma1112 commented Mar 11, 2025

Description

Endpoints for the kernel interrupt and kernel restart endpoints miss the "/api" prefix that all other endpoints have in these files.
As a result when one auto-generates code from the swagger files, the functionalities that call these endpoints will not work.

Reproduce

One can already see that the endpoints need the /api prefix by e.g. creating a kernel with

POST /api/kernels

capturing the returned session ID and then calling the following endpoints. The first two work, the last two does not work.

POST /api/kernels<kernel_id>/interrupt
POST /api/kernels<kernel_id>/restart

POST /kernels<kernel_id>/interrupt
POST /kernels<kernel_id>/interrupt

Expected behavior

swagger files should have the proper api path

Context

See also this issue that reports different defects in the swagger files. #404

@ma1112 ma1112 added the bug label Mar 11, 2025
ma1112 pushed a commit to ma1112/kernel_gateway that referenced this issue Mar 11, 2025
@ma1112 ma1112 linked a pull request Mar 11, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant