Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds support for using official 123 网盘 (123pan) app credentials via environment bindings, and adjusts the UI to allow “server-side/official credentials” mode for the 123 网盘 driver.
Changes:
- Add
cloud123_uid/cloud123_keybindings to Wrangler configs and runtime env injection. - Update the 123 网盘 OAuth/login flow to optionally use server-provided credentials and send JSON payloads.
- Add a new
/approute and adjust the frontend driver-selection behavior to no longer forbid server-side mode for123cloud_go.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| wrangler.jsonc | Adds cloud123_uid/cloud123_key to Worker vars. |
| wrangler.example.jsonc | Adds example cloud123_uid/cloud123_key entries. |
| src/index.ts | Adds cloud123_* to bindings and introduces /app redirect route. |
| src/driver/123cloud_oa.ts | Uses server env creds when server_use is enabled; switches request to JSON. |
| src/basic.ts | Injects cloud123_* from process.env for node dev server. |
| public/static/event.js | Allows server-side mode selection for 123cloud_go. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Pikachu Ren <40362270+PIKACHUIM@users.noreply.github.com>
jyxjjj
approved these changes
Mar 31, 2026
Suyunmeng
approved these changes
Mar 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
add(driver): 123 official app