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.

Sign up for GitHub

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

Jump to bottom

enable/disable interface HTTP status code isn't consistent with the rest of the endpoints #128

Open

viniarck opened this issue Apr 24, 2023 · 0 comments

Labels

priority_low tech_debt

Member

viniarck commented Apr 24, 2023

switches, links are using HTTP status code 201 for enable/disable, but interfaces are using 200

POST v3/links/<link_id>/enable -> 201
POST v3/switches//enable -> 201
POST v3/interfaces/<interface_enable_id>/enable -> 200

201 is usually for new resource creation when it doesn't exist, I'd say all enable/disable should return 200 when succeeding.

viniarck added priority_low tech_debt labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment