-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update success code #345
Update success code #345
Conversation
@congwang09 for successful place_connection, PCE still returns a '200', not '201' |
Pull Request Test Coverage Report for Build 11524874793Details
💛 - Coveralls |
Sounds good. Updated to use generic checking, e.g., 200-299 are all success. |
@congwang09 Could you also produce 402, 409, 410 codes in connection_handler.py?
Now they all return 400. 400 is returned when the body is not proper json, returned by Swagger already. |
Relates to: #340
Update the success code. It seems that all the 4xx codes will be propagated from pce.