search

0xasahi / neogroup

Donation: https://ko-fi.com/neogroup
https://neogrp.club/
MIT License
12 stars 5 forks source link

feat: keep redirect_uri same #11

Closed OmmyZhang closed 1 year ago

OmmyZhang commented 1 year ago

ref: #10

0xasahi commented 1 year ago

这部分改成 settings 是不是反而不灵活了？

OmmyZhang commented 1 year ago

这部分改成 settings 是不是反而不灵活了？

没有呀，创建app的时候用的"redirect_uris": settings.REDIRECT_URIS，用别的地址本来也登录不了。具体见issue。

0xasahi commented 1 year ago

喔，不对，此处不能这么改，因为用户可能来自各种各样不同的实例，所以要动态获取实例的 host 等信息

OmmyZhang commented 1 year ago

不是呀，这里的get_host()是neogroup的host不是登录用实例的host，登录用实例是传进来的参数login_domain，你再看看呢？

0xasahi commented 1 year ago

阿，确实，好像是我理解的问题，我晚上仔细看一下 issue 里提到的问题

OmmyZhang commented 1 year ago

刚才倒是想到一种可能，neogroup要支持多个域名（比如应对被墙什么的）。这样的话需要先配置好所有域名以正确创建app，然后登录的时候动态获取用户访问的哪个host，并且做个检查确认是不是在列表里。

如果想用这个方案的话我另外提个pr。