NKUST-ITC / NKUST-AP-API

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

最新消息 新增/修改/刪除 文件 #37

Closed takidog closed 5 years ago

takidog commented 5 years ago

35

/news/announcements 會有需要管理員執行,在開啟伺服器之前 可以透過環境變數去設定管理員

輸入校務系統的帳號

export NEWS_ADMIN=c1107111111

如有多個管理員

export NEWS_ADMIN=1105111111;1104121212

使用;去分隔


關於如何驗證

可以把 /news/announcements 當作是校務系統中的服務 一樣是透過/oauth/token去登入

之後我們會驗證JWT中的username 確認是否有存在於NEWS_ADMIN

abc873693 commented 5 years ago

目前有打算加入驗證機制?

takidog commented 5 years ago

目前有打算加入驗證機制?

已經有驗證了

驗證透過校務系統,在後端再加入管理員的名單

abc873693 commented 5 years ago

這個功能對應的 route 應該是 /news/announcements route 設計上應該要基於這下面 需要確認一下

abc873693 commented 5 years ago

驗證透過校務系統,在後端再加入管理員的名單

38 缺少這個問題所需要的參數

之後要再補上去

takidog commented 5 years ago

驗證透過校務系統,在後端再加入管理員的名單

38 缺少這個問題所需要的參數

之後要再補上去

後端驗證跟一般使用者的驗證沒有不同

這個功能對應的 route 應該是 /news/announcements route 設計上應該要基於這下面 需要確認一下

可以,新增上去了

takidog commented 5 years ago

204 >> 200 的文件更新 在登入的時候會有isAdmin欄位 @abc873693 再麻煩看了

abc873693 commented 5 years ago

@takidog 我需要完整的驗證機制 上面敘述不是很清楚 很容易搞混

takidog commented 5 years ago

登入

使用者用學校帳號密碼登入 >>> 後端向學校登入,登入成功並返回JWT

操作NEWS

JWT >> 後端確認他的帳號是否是管理員 >>允許操作

abc873693 commented 5 years ago

登入

使用者用學校帳號密碼登入 >>> 後端向學校登入,登入成功並返回JWT

操作NEWS

JWT >> 後端確認他的帳號是否是管理員 >>允許操作

你這邊的 NEWS 應該是 /news/announcements

最後是 哪裡可以設定管理員?

takidog commented 5 years ago

登入

使用者用學校帳號密碼登入 >>> 後端向學校登入,登入成功並返回JWT

操作NEWS

JWT >> 後端確認他的帳號是否是管理員 >>允許操作

你這邊的 NEWS 應該是 /news/announcements

是的沒錯 忘記有news/school的存在了

abc873693 commented 5 years ago

登入

使用者用學校帳號密碼登入 >>> 後端向學校登入,登入成功並返回JWT

操作NEWS

JWT >> 後端確認他的帳號是否是管理員 >>允許操作

你這邊的 NEWS 應該是 /news/announcements

是的沒錯 忘記有news/school的存在了

把相關資訊補在 最上面

最後是 哪裡可以設定管理員?

takidog commented 5 years ago

登入

使用者用學校帳號密碼登入 >>> 後端向學校登入,登入成功並返回JWT

操作NEWS

JWT >> 後端確認他的帳號是否是管理員 >>允許操作

你這邊的 NEWS 應該是 /news/announcements

是的沒錯 忘記有news/school的存在了

把相關資訊補在 最上面

最後是 哪裡可以設定管理員?

好 我會編輯在主樓

使用環境變數設定管理員 再另一份PR中有修改README.md 在README.md有說明

export NEWS_ADMIN=1105111111

如有多個管理員

export NEWS_ADMIN=1105111111;1104121212

使用;去分隔

abc873693 commented 5 years ago

@vn7n24fzkq 你確認一下 可不可以

vn7n24fzkq commented 5 years ago

看起來很棒沒什麼問題 ,只是用環境變數似乎不是個好辦法,merge 之後加個增強的標籤如何 ? 也許有更好的辦法,不過這個功能這樣算是完成了