Skip to content

fix auth exposure and release workflows#121

Closed
phenix3443 wants to merge 14 commits into
perfect-panel:masterfrom
phenix3443:master
Closed

fix auth exposure and release workflows#121
phenix3443 wants to merge 14 commits into
perfect-panel:masterfrom
phenix3443:master

Conversation

@phenix3443
Copy link
Copy Markdown
Contributor

@phenix3443 phenix3443 commented Apr 15, 2026

Summary

  • hide unsupported or incompletely configured auth methods from the public global config response, and reject disabled or incomplete OAuth login requests
  • avoid blocking server startup when the GeoIP database is missing by continuing without GeoIP support
  • update GitHub Actions to use newer checkout/login actions, publish release images to GHCR, and trigger the matching ppanel packaging workflow after releases

Validation

  • no additional local validation was run for this metadata update

@phenix3443 phenix3443 changed the title [codex] sync fork master after merging development work fix auth exposure and release workflows Apr 15, 2026
@phenix3443 phenix3443 marked this pull request as ready for review April 15, 2026 07:44
@web-ppanel
Copy link
Copy Markdown
Contributor

Thanks for the contribution! The code changes here are solid, but this PR mixes several unrelated concerns:

  1. OAuth config validation + Telegram Redis key fix
  2. GeoIP non-blocking startup
  3. Release workflow migration from Docker Hub → GHCR
  4. AI tooling files (AGENT.md, CLAUDE.md, BRANCH_STRATEGY.md, CONTRIBUTING.md)

Could you please split these into separate PRs so we can review and merge them individually? The workflow/registry change in particular is a breaking change for existing users and needs to be evaluated separately.

Closing for now — please reopen individual PRs for each concern.

@web-ppanel web-ppanel closed this May 10, 2026
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.

2 participants