-
Notifications
You must be signed in to change notification settings - Fork 0
Labels
good first issueGood for newcomersGood for newcomers
Description
GET /users や GET /users/me が呼び出されたときに走る処理は、service/user.go の User interface によって実装されている。しかし、この interface の実装は service/v1 にある。他の処理は service/v2 で実装されたものに移行しているため、この実装も移行したい。この User interface と同じような処理を行うものとして、 OIDCV2 interface が定義されており、この実装が service/v2/oidc.go で OIDC 構造体で提供されているため、これを使うように変更すればよい。ただし、 #1103 で bot を弾くためのパラメータが導入されているため、これを OIDCV2 にも導入する必要がある。
具体的な作業
service.OIDCV2のGetActiveUsersが、botを含むかどうかの引数を取るように interface 定義を変える- 1 で定義した interface に合うように
service/v2.OIDCに定義されたGetActiveUsersの実装を修正する - handler/users.go の
User構造体にservice.OIDCV2を持たせ、GetMeとGetUsersでそれを使ってメソッドを呼ぶようにする。
テストの修正も必要
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomers