[PR #771] [MERGED] fix: 修复自定义认证密码功能异常 #752

Closed
opened 2026-02-26 21:32:48 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/dreamhunter2333/cloudflare_temp_email/pull/771
Author: @dreamhunter2333
Created: 11/23/2025
Status: Merged
Merged: 11/23/2025
Merged by: @dreamhunter2333

Base: mainHead: feature/email


📝 Commits (1)

  • 0a0f5a4 fix: 修复自定义认证密码功能异常的问题

📊 Changes

3 files changed (+6 additions, -2 deletions)

View changed files

📝 CHANGELOG.md (+4 -0)
📝 frontend/src/api/index.js (+1 -1)
📝 worker/src/worker.ts (+1 -1)

📄 Description

User description

Summary

修复自定义认证密码功能的两个关键问题:

  • 修复前端认证弹窗无法显示的问题(属性名错误)
  • 修复 /open_api/settings 接口被认证拦截导致应用无法启动的问题

Changes

  • frontend/src/api/index.js:44 - 修正属性引用 openSettings.value.authopenSettings.value.needAuth
  • worker/src/worker.ts:55 - 为 /open_api 路径添加认证豁免,避免循环依赖
  • CHANGELOG.md - 添加 v1.1.1 版本更新日志

Test Plan

  • 代码审查通过
  • 测试自定义认证密码功能是否正常工作
  • 验证 /open_api/settings 接口可正常访问
  • 验证认证弹窗在需要时正常显示

🤖 Generated with Claude Code


PR Type

Bug fix


Description

  • Fixed incorrect property reference in frontend API handling.

  • Added authentication exemption for /open_api path in backend.

  • Updated changelog with details of the fixes.


Changes walkthrough 📝

Relevant files
Bug fix
worker.ts
Add authentication exemption for `/open_api` path               

worker/src/worker.ts

  • Added authentication exemption for /open_api path.
  • Prevented custom authentication from blocking /open_api/settings.
  • +1/-1     
    index.js
    Fix incorrect property reference in frontend API                 

    frontend/src/api/index.js

  • Fixed incorrect property reference for openSettings.value.auth.
  • Updated to use openSettings.value.needAuth for authentication check.
  • +1/-1     
    Documentation
    CHANGELOG.md
    Update changelog with v1.1.1 details                                         

    CHANGELOG.md

  • Added v1.1.1 entry with details of fixes.
  • Documented frontend and backend authentication fixes.
  • +4/-0     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.

  • 🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

    ## 📋 Pull Request Information **Original PR:** https://github.com/dreamhunter2333/cloudflare_temp_email/pull/771 **Author:** [@dreamhunter2333](https://github.com/dreamhunter2333) **Created:** 11/23/2025 **Status:** ✅ Merged **Merged:** 11/23/2025 **Merged by:** [@dreamhunter2333](https://github.com/dreamhunter2333) **Base:** `main` ← **Head:** `feature/email` --- ### 📝 Commits (1) - [`0a0f5a4`](https://github.com/dreamhunter2333/cloudflare_temp_email/commit/0a0f5a4829d9cfd94360ab6b61d657af59dcbcd4) fix: 修复自定义认证密码功能异常的问题 ### 📊 Changes **3 files changed** (+6 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+4 -0) 📝 `frontend/src/api/index.js` (+1 -1) 📝 `worker/src/worker.ts` (+1 -1) </details> ### 📄 Description ### **User description** ## Summary 修复自定义认证密码功能的两个关键问题: - 修复前端认证弹窗无法显示的问题(属性名错误) - 修复 `/open_api/settings` 接口被认证拦截导致应用无法启动的问题 ## Changes - **frontend/src/api/index.js:44** - 修正属性引用 `openSettings.value.auth` → `openSettings.value.needAuth` - **worker/src/worker.ts:55** - 为 `/open_api` 路径添加认证豁免,避免循环依赖 - **CHANGELOG.md** - 添加 v1.1.1 版本更新日志 ## Test Plan - [x] 代码审查通过 - [ ] 测试自定义认证密码功能是否正常工作 - [ ] 验证 `/open_api/settings` 接口可正常访问 - [ ] 验证认证弹窗在需要时正常显示 🤖 Generated with [Claude Code](https://claude.ai/code) ___ ### **PR Type** Bug fix ___ ### **Description** - Fixed incorrect property reference in frontend API handling. - Added authentication exemption for `/open_api` path in backend. - Updated changelog with details of the fixes. ___ ### **Changes walkthrough** 📝 <table><thead><tr><th></th><th align="left">Relevant files</th></tr></thead><tbody><tr><td><strong>Bug fix</strong></td><td><table> <tr> <td> <details> <summary><strong>worker.ts</strong><dd><code>Add authentication exemption for `/open_api` path</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; </dd></summary> <hr> worker/src/worker.ts <li>Added authentication exemption for <code>/open_api</code> path.<br> <li> Prevented custom authentication from blocking <code>/open_api/settings</code>. </details> </td> <td><a href="https://github.com/dreamhunter2333/cloudflare_temp_email/pull/771/files#diff-c007030a206d7e4e2a4879ba673551e7633ad914b67cfac204096edced3f80a2">+1/-1</a>&nbsp; &nbsp; &nbsp; </td> </tr> <tr> <td> <details> <summary><strong>index.js</strong><dd><code>Fix incorrect property reference in frontend API</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; </dd></summary> <hr> frontend/src/api/index.js <li>Fixed incorrect property reference for <code>openSettings.value.auth</code>.<br> <li> Updated to use <code>openSettings.value.needAuth</code> for authentication check. </details> </td> <td><a href="https://github.com/dreamhunter2333/cloudflare_temp_email/pull/771/files#diff-80f7e17c8e1f299b7316ce792286ebcaab67ed3afc050e3c5019ed33ab61b34b">+1/-1</a>&nbsp; &nbsp; &nbsp; </td> </tr> </table></td></tr><tr><td><strong>Documentation</strong></td><td><table> <tr> <td> <details> <summary><strong>CHANGELOG.md</strong><dd><code>Update changelog with v1.1.1 details</code>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; </dd></summary> <hr> CHANGELOG.md <li>Added v1.1.1 entry with details of fixes.<br> <li> Documented frontend and backend authentication fixes. </details> </td> <td><a href="https://github.com/dreamhunter2333/cloudflare_temp_email/pull/771/files#diff-06572a96a58dc510037d5efa622f9bec8519bc1beab13c9f251e97e657a9d4ed">+4/-0</a>&nbsp; &nbsp; &nbsp; </td> </tr> </table></td></tr></tr></tbody></table> ___ > <details> <summary> Need help?</summary><li>Type <code>/help how to ...</code> in the comments thread for any questions about PR-Agent usage.</li><li>Check out the <a href="https://qodo-merge-docs.qodo.ai/usage-guide/">documentation</a> for more information.</li></details> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
    kerem 2026-02-26 21:32:48 +03:00
    Sign in to join this conversation.
    No milestone
    No project
    No assignees
    1 participant
    Notifications
    Due date
    The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

    No due date set.

    Dependencies

    No dependencies set.

    Reference
    starred/cloudflare_temp_email#752
    No description provided.