fix: oauth2 redirect url not consistent

[uubulb]

1. strings.ToLower 延后到更新数据库条目时调用
2. callback 和 bind api 使用对应的 redirect url

Fix #928

[naiba]

path一致还是完全一致，能传不同query吗

…

On Tue, Dec 31, 2024 at 17:46 UUBulb ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In cmd/dashboard/controller/oauth2.go <#930 (comment)>: > @@ -91,7 +91,7 @@ func exchangeOpenId(c *gin.Context, o2confRaw *model.Oauth2Config, provider stri return "", singleton.Localizer.ErrorT("invalid state key") } - o2conf := o2confRaw.Setup(getRedirectURL(c, provider, rTypeLogin)) + o2conf := o2confRaw.Setup(getRedirectURL(c, provider, typ)) cf access 是要求 authorization redirect uri 和申请 token 的 redirect uri 一致 — Reply to this email directly, view it on GitHub <#930 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG7DUMIJ7PE2OFLB3AR26332IJRYJAVCNFSM6AAAAABUMOYT4CVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMRWGMZDGNBQGI> . You are receiving this because you commented.Message ID: ***@***.***>

[uubulb]

path一致还是完全一致，能传不同query吗

不行 provider query 必须带上
好像只有github才是前缀匹配

[naiba]

加query也不行的吗，那只能加cookie了呀

…

On Tue, Dec 31, 2024 at 17:50 UUBulb ***@***.***> wrote: path一致还是完全一致，能传不同query吗 不行 provider query 必须带上 好像只有github才是前缀匹配 — Reply to this email directly, view it on GitHub <#930 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG7DUMOT3BIWQAQEQ6EDYST2IJSH5AVCNFSM6AAAAABUMOYT4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRWGI4TOMJTGM> . You are receiving this because you commented.Message ID: ***@***.***>

[naiba]

cache的state加个行为，对应行为操作

[naiba]

你直接改下？应该只是改下api，前端删些东西就好

[uubulb]

cache的state加个行为，对应行为操作

url只用一个吗

[naiba]

是的 那个cachekey的value加个action得了，绑定就绑定，不绑定就登录，前端相关逻辑和接口删除

…

On Tue, Dec 31, 2024 at 18:05 UUBulb ***@***.***> wrote: cache的state加个行为，对应行为操作 url只用一个吗 — Reply to this email directly, view it on GitHub <#930 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG7DUMOC5NR7G6BD2C6DCLL2IJT7HAVCNFSM6AAAAABUMOYT4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRWGMYDOOBYHA> . You are receiving this because you commented.Message ID: ***@***.***>

[naiba]

直接跳转过去

…

On Tue, Dec 31, 2024 at 18:41 William Liu ***@***.***> wrote: 是的 那个cachekey的value加个action得了，绑定就绑定，不绑定就登录，前端相关逻辑和接口删除 On Tue, Dec 31, 2024 at 18:05 UUBulb ***@***.***> wrote: > cache的state加个行为，对应行为操作 > > url只用一个吗 > > — > Reply to this email directly, view it on GitHub > <#930 (comment)>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AG7DUMOC5NR7G6BD2C6DCLL2IJT7HAVCNFSM6AAAAABUMOYT4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRWGMYDOOBYHA> > . > You are receiving this because you commented.Message ID: > ***@***.***> >

[uubulb]

先吃饭， 等一会写