更新API文件 (校車)

NKUST-ITC / NKUST-AP-API

NKUST-AP-v3-完善計畫

MIT License

7 stars 5 forks source link

Closed takidog closed 5 years ago

takidog commented 5 years ago

移除404 新增403，當非[建工/燕巢]校區學生時回應

abc873693 commented 5 years ago

PR 合併錯地方了文件都統一送到gh-pages

takidog commented 5 years ago

重開，更改合併的分支

abc873693 commented 5 years ago

移除404的理由？

abc873693 commented 5 years ago

那畢業生會回應403嗎？

abc873693 commented 5 years ago

另外沒有issue提到這個問題我目前在測試 server 一個畢業生帳號回應401 一個高第一帳號回應403

takidog commented 5 years ago

移除404的理由？

404應該是給使用者找不到伺服器的route 不是表示後端找不到學校的資源才是

abc873693 commented 5 years ago

@vn7n24fzkq 你覺得咧？

takidog commented 5 years ago

另外沒有issue提到這個問題我目前在測試 server 一個畢業生帳號回應401 一個高第一帳號回應403

高應畢業生學校給出來的反應是登入失敗不同校區是找不到資料

vn7n24fzkq commented 5 years ago

這需要討論一下，我自己會希望可以把目前學校混亂的 API 透過這個 API server 做一個一致性的分類，讓其他有串接的開發者可以不用去管學校伺服器到底發生甚麼事情只要專心在這個 API 介面上就好 @takidog 你開發起來的經驗這是可行的嗎

takidog commented 5 years ago

這需要討論一下，我自己會希望可以把目前學校混亂的 API 透過這個 API server 做一個一致性的分類，讓其他有串接的開發者可以不用去管學校伺服器到底發生甚麼事情只要專心在這個 API 介面上就好 @takidog 你開發起來的經驗這是可行的嗎

讓其他串的人可以不用管學校伺服器，但這個前提要我們了解學校所有回傳的東西會比較精準的回傳錯誤

那這邊應該是404的問題如果我們要定義一個學校的問題，應該找在規範內的http status code回應沒有相對應的code，再來找其他status code替代

不然我是一個外面的人來串，看到404我第一直覺是會覺得我請求的url是不是錯了

abc873693 commented 5 years ago

根據上述問題這個404我沒問題但是其他問題需要再開issue討論

abc873693 commented 5 years ago

但實際實作功能還需要再檢查

vn7n24fzkq commented 5 years ago

@taki 那我覺得ok 404 移除 ,thx