[INFRAUTILS-46] diagstatus error code is 418 instead of 503 Created: 21/Aug/18 Updated: 04/Sep/18 Resolved: 04/Sep/18 |
|
| Status: | Resolved |
| Project: | infrautils |
| Component/s: | diagstatus |
| Affects Version/s: | None |
| Fix Version/s: | Oxygen-SR4, Fluorine-SR1, Neon |
| Type: | Improvement | Priority: | Medium |
| Reporter: | Tim Rozet | Assignee: | Michael Vorburger |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
diagstatus currently returns a joke return code that should not be used. 503 makes more sense as it is "Service Unavailable" which can be do for maintenance or some other reason. |
| Comments |
| Comment by Michael Vorburger [ 22/Aug/18 ] |
|
This is from |
| Comment by Michael Vorburger [ 22/Aug/18 ] |
|
see RFC-2324 & RFC-7168 for the meaning of original HTTP return code 418 (now changed to 503). |
| Comment by Michael Vorburger [ 22/Aug/18 ] |
|
k.faseela I doubt you guys (E//) downstream hard-coded 418 somewhere that needs changing to 503 ? jluhrsen similar question for upstream CSIT - you probably just check the /diagstatus/ URL for non-200? So hopefully all good and no impat to change this here. Please shout if this needs more careful coordination. Teapots, be gone. |
| Comment by Faseela K [ 23/Aug/18 ] |
|
vorburger : We are not using diagstatus web downstream. |