search

NKUST-ITC / NKUST-AP-API

NKUST-AP-v3-完善計畫
MIT License
7 stars 5 forks source link

請假功能 /leave/submit/info 畢業生回傳問題 #48

Closed takidog closed 5 years ago

takidog commented 5 years ago

文件上有標403 User can't user this feature

當初沒有實作出來

有需要修改其他回傳值嗎

abc873693 commented 5 years ago

理論上不用 但我一直覺得類似這種是不是要有一定格式 message 和 error code 用json表示

takidog commented 5 years ago

理論上不用 但我一直覺得類似這種是不是要有一定格式 message 和 error code 用json表示

目前看起來，沒有其他狀態會跟這個一個api上的403衝突 應該可以先只回應403

takidog commented 5 years ago

以防萬一 還是定義一下 http status code : 403

{
    "title": "403 Forbidden",
    "code": 400
}

這樣可以嗎? @abc873693

abc873693 commented 5 years ago

tile 改 message比較妥

takidog commented 5 years ago

tile 改 message比較妥

那個是falcon預設的回應方式 還是要另外用一個有message的?

也可以預設的回應調整成這樣

{
    "title": "403 Forbidden",
    "description": "400, graduate can't use this feature ",
    "code": 400
}
abc873693 commented 5 years ago

其他api 也是這樣？

takidog commented 5 years ago

是 除了200 其他的錯誤都是用falcon內建的回應 有多種錯誤狀態的也是用這

abc873693 commented 5 years ago

tile 改 message比較妥

那個是falcon預設的回應方式 還是要另外用一個有message的?

也可以預設的回應調整成這樣

{
    "title": "403 Forbidden",
    "description": "400, graduate can't use this feature ",
    "code": 400
}

這樣這個OK

abc873693 commented 5 years ago

50