Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

支持通过 OAuth 登录 Littleskin #3491

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Conversation

Glavo
Copy link
Member

@Glavo Glavo commented Dec 9, 2024

No description provided.

@3gf8jv4dv
Copy link
Contributor

终于来了

可以参与这个 PR 的测试吗?该怎么测试?

@Glavo Glavo marked this pull request as draft December 18, 2024 12:35
@cqw-acq
Copy link

cqw-acq commented Dec 24, 2024

终于来了

可以参与这个 PR 的测试吗?该怎么测试?

我也想测试:)

@tnqzh123
Copy link

@3gf8jv4dv @cqw-acq

Actions 中有构建好的包

@shimoranla
Copy link

yushijinhun/authlib-injector#268

Authlib-Injector 的新协议(确信

@tnqzh123
Copy link

tnqzh123 commented Jan 21, 2025

@shimoranla

yushijinhun/authlib-injector#268

Authlib-Injector 的新协议(确信

实际上这个 PR 就是基于 Yggdrasil Connect 实现的,但是不包含服务发现(限定 LittleSkin),部分实现和规范不符(主要是读取 selectedProfile 声明里的 properties 字段的问题,规范里 selectedProfile 声明中不包含角色属性,但是 LittleSkin 先前的实现中有包含,主要是我最初制定规范时考虑不周,后期临时改规范导致的)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants