Skip to content

Copilot/update mc multiplayer skin login#704

Open
Seele-Vollerei32 wants to merge 6 commits into
natfrp:masterfrom
Seele-Vollerei32:copilot/update-mc-multiplayer-skin-login
Open

Copilot/update mc multiplayer skin login#704
Seele-Vollerei32 wants to merge 6 commits into
natfrp:masterfrom
Seele-Vollerei32:copilot/update-mc-multiplayer-skin-login

Conversation

@Seele-Vollerei32

Copy link
Copy Markdown
Contributor

This pull request adds documentation to help users set up and use third-party skin server (皮肤站) authentication for Minecraft LAN and online servers. The changes clarify requirements for using skin site accounts and provide step-by-step instructions for configuring server-side authentication with authlib-injector.

Skin server (皮肤站) authentication improvements:

  • Added a tip section in app/mc.md explaining that all players (including the host) must log in with the same skin server account and use a launcher that supports third-party authentication, with guidance on how to set this up.
  • Added a detailed guide in offtopic/mc-java-server.md on enabling skin server authentication for online-mode servers using authlib-injector, including download instructions, API configuration, JVM argument setup (with special note for Forge/NeoForge), and ensuring online-mode=true.

@Seele-Vollerei32 Seele-Vollerei32 marked this pull request as ready for review June 15, 2026 02:13
Copilot AI review requested due to automatic review settings June 15, 2026 02:13
@Seele-Vollerei32 Seele-Vollerei32 marked this pull request as draft June 15, 2026 02:13

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds documentation for using a third-party skin site (Yggdrasil) to allow “online-mode=true” verification, and clarifies launcher requirements when joining via skin site accounts.

Changes:

  • Document server-side setup using authlib-injector to redirect authentication to a skin site Yggdrasil API.
  • Add an in-app guide tip describing what a skin site account is and launcher requirements for third-party login.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
offtopic/mc-java-server.md Adds a new section explaining authlib-injector/Yggdrasil-based auth for Java servers.
app/mc.md Adds a tip block explaining skin site accounts and launcher setup requirements.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread offtopic/mc-java-server.md Outdated
Comment thread offtopic/mc-java-server.md Outdated
Comment thread app/mc.md Outdated
@Seele-Vollerei32 Seele-Vollerei32 marked this pull request as ready for review June 15, 2026 02:29
@Paper-white

Copy link
Copy Markdown

必要性不大,且服务端配置那部分写的很有问题
部分 forge/neoforge 服务端存在一个 “start.jar” 这个情况下直接传递参数会丢掉

顶多在 “所有人都登录皮肤站的账号,不需辅助 Mod” 这个位置加上一个Littleskin手册的链接作为引用推荐就行了,其他的没必要

@natfrp-bot

natfrp-bot commented Jun 15, 2026

Copy link
Copy Markdown
Member

我对 PCL2 部分放爱发电链接有点疑问,这个启动器没有官网或者单独的 GH 项目么? 🤔

如果实在没有,也许可以用这个 https://ifdian.net/p/0164034c016c11ebafcb52540025c377

@Paper-white

Paper-white commented Jun 15, 2026

Copy link
Copy Markdown

我对 PCL2 部分放爱发电链接有点疑问,这个启动器没有官网或者单独的 GH 项目么? 🤔

讲个笑话,真的没有

作为Littleskin方手册维护者,我认为这里内容都有些多余,只需要保留皮肤站是什么,然后引用一个Littleskin的手册就够了

在这边详细的写如何配置反而加大了后期维护的工作量

@fakeboboliu

Copy link
Copy Markdown
Contributor

我对 PCL2 部分放爱发电链接有点疑问,这个启动器没有官网或者单独的 GH 项目么? 🤔

如果实在没有,也许可以用这个 https://ifdian.net/p/0164034c016c11ebafcb52540025c377

设定上来说官网是这个 https://meloong.com/PCL

但好像并没有操作性

@Paper-white

Copy link
Copy Markdown

在这里写既没必要,又写不清楚比如
使用使用三方鉴权的好处/坏处
需要注意的事项 ,配置之后可能带来的影响

不写清楚这些东西,就盲目的让用户去配置三方验证,只会徒增麻烦事
就算真的需要扩展解释也只需要直接引用跳转LS的文档就可以了

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.

6 participants