| << Back to previous view |
|
[QB-3578] Send status reason text along with status code in pre-queue evaluation response
|
|
| Status: | Resolved |
| Project: | QuickBuild |
| Component/s: | None |
| Affects Version/s: | 10.0.13 |
| Fix Version/s: | 11.0.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Robin Shen | Assigned To: | Unassigned |
| Resolution: | Fixed | Votes: | 3 |
| Remaining Estimate: | Unknown | Time Spent: | Unknown |
| Original Estimate: | Unknown | ||
| Comments |
| Comment by Dong Van Thang [ 01/Jun/20 06:38 AM ] |
|
Hope you will implement it soon.
Thank you! |
| Comment by Robin Shen [ 01/Jun/20 11:38 PM ] |
| Should be available by end of this week |
| Comment by Robin Shen [ 05/Jun/20 12:20 PM ] |
|
This has now been fixed in 10.0.16. To send status reason together with status code, return a string in pre-queue script something like below:
"500:Some special server error" The number before colon is status code, and text after colon represents status reason. Since there is no standard way to return a customized status text in http procotol, the status reason is carried in http response via header "statusReason" |
| Comment by Dong Van Thang [ 08/Jun/20 08:26 AM ] |
|
Oh, Sorry, why don't we return text message via body????
like return Response.status(Integer.parseInt(beforeColon)).entity(StringUtils.substringAfter(result.getRejectReason(), ":")).build();? Thank you! |
| Comment by Robin Shen [ 08/Jun/20 11:10 PM ] |
| Because it will be silently dropped, as body is not expected to convey any messages in case there are http errors. |
| Comment by Dong Van Thang [ 24/Jul/20 04:47 AM ] |
|
Dear QuickBuild team,
After we have done implement QB10, we realize that: 1. Many users do not check header contents. 2. lt is more common to give reject reason in the body. So, we would like you to improve this problem, return text message via response's body. What do you think so? Please kindly give me your idea! Thank you so much! |
| Comment by Robin Shen [ 24/Jul/20 10:59 PM ] |
| I am reopening this issue and will investigate further. |
| Comment by Dong Van Thang [ 27/Jul/20 02:55 AM ] |
| Thank you so much! |
| Comment by Dong Van Thang [ 28/Aug/20 02:22 AM ] |
| Thank you so much for this improvement. |
| Comment by Robin Shen [ 28/Aug/20 04:04 AM ] |
| Sorry this is a mistake. It is not implemented yet. |
| Comment by DPI SVMC [ 26/Nov/20 02:40 AM ] |
| Please consider about this improvement again. Thank you! |
| Comment by Dong Van Thang [ 27/Jan/21 08:47 AM ] |
|
Hello Mr. Robin Shen,|
Please consider about returning text message via response's body. Thank you so much! |
| Comment by Robin Shen [ 27/Jan/21 01:09 PM ] |
| Will get it fixed in QB 11 which will be shipped end of March |