Closed takidog closed 5 years ago
404 代表可能該使用者 可能校務系統找不到該資料 例如學生資料 畢業生是找不到學籍資料 需要這個 status code 或是這種情況有更好的 status code 有些status code 是之前 討論的 必然有他的需求
500系列的錯誤要明確定義 本身該功能有錯誤 不能用其他功能去再次驗證 會多增加送要求的次數 也不能做不出來 就把它給刪除 基本還是得定義出來 未來開issue看有沒有人能做出來 實作可以先合併 未來有做出來 前端才不會又需要改
v2先前有building這個欄位我需要確認 但我偏向如果抓不到 先讓他是空的也是OK 日後有抓到就不用再改前端
這個功能當初定義是製作文件時功能還在 現在貌似學校 拿掉此功能 我偏向 先註解無法實作 學校會不會恢復 過一段時間再討論 另開issue說明這個問題
其他沒啥問題 @vn7n24fzkq 你看一下 東西要打太多 我直接打中文比較清楚QQ
[DELETE] /oauth/token [DELETE] /oauth/token/all 這兩個留著是不是比較好 畢竟不是只給單一平台 token本來就要存多個
[DELETE] /oauth/token [DELETE] /oauth/token/all 這兩個留著是不是比較好 畢竟不是只給單一平台 token本來就要存多個
補回去了 會在後續修改登入邏輯,來達成這項功能
3
這PR只處理User部分和Auth部分
移除API
[DELETE] /oauth/token [DELETE] /oauth/token/all 如果要實踐這2個功能 需要再多儲存使用者目前存活的token等等,在每次登入除了驗證完JWT 還要多做這token是不是已經被刪除了 太過多餘且JWT能做到過期,所以移除這2個API
Status code
404 User 部分的status code 都有確定的url了哪來的404
504 學校就算系統維護,也會回200 也沒有太多資料能判斷學校的回覆內容 如果要判斷學校伺服器狀態,後續從server/info獲得 發生錯誤使用500 / 503 才是
格式
course table 格式 目前課表沒有"建築物“ 所以移除,只留下room 如後續學校有更新,那再來更新建築物
畢業門檻 目前從webap 畢業門檻沒辦法顯示 (學校前端問題) 查詢後發現校外實習沒有出現,目前先留下個人資料以及英文門檻
GET Query String
查詢教室課表(empty-room) 不需要校區參數,故移除