[PR #756] [MERGED] fix: ASN blacklist not working due to missing asnBlacklist field #745

New issue

Closed

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

kerem commented

2026-02-26 21:32:47 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/dreamhunter2333/cloudflare_temp_email/pull/756
Author: @dreamhunter2333
Created: 11/3/2025
Status: ✅ Merged
Merged: 11/3/2025
Merged by: @dreamhunter2333

Base: main ← Head: feature/email

📝 Commits (1)

cdd73f2 fix: ASN blacklist not working due to missing asnBlacklist field

📊 Changes

2 files changed (+14 additions, -12 deletions)

View changed files

📝 worker/src/admin_api/ip_blacklist_settings.ts (+2 -1)
📝 worker/src/ip_blacklist.ts (+12 -11)

📄 Description

User description

Fix getIpBlacklistSettings() to include asnBlacklist field in return value
Add case-insensitive flag support for regex patterns in ASN matching
Refactor IP blacklist check logic for better code organization

🤖 Generated with Claude Code

PR Type

Bug fix, Enhancement

Description

Fix missing asnBlacklist field in IP blacklist settings.
Add case-insensitive flag support for regex patterns.
Refactor IP blacklist logic for better organization.
Improve handling of IP and ASN blacklist checks.

Changes walkthrough 📝

Relevant files

Bug fix

ip_blacklist_settings.ts

Add `asnBlacklist` field to IP blacklist settings

worker/src/admin_api/ip_blacklist_settings.ts

Add asnBlacklist field to default settings.

Ensure asnBlacklist is included in returned settings.

+2/-1

Enhancement

ip_blacklist.ts

Enhance IP and ASN blacklist logic and regex handling

worker/src/ip_blacklist.ts

Add case-insensitive regex flag for blacklist checks.

Include asnBlacklist in settings retrieval logic.

Refactor IP blacklist check to handle empty lists.

Improve ASN blacklist handling in IP checks.

+12/-11

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/756 **Author:** [@dreamhunter2333](https://github.com/dreamhunter2333) **Created:** 11/3/2025 **Status:** ✅ Merged **Merged:** 11/3/2025 **Merged by:** [@dreamhunter2333](https://github.com/dreamhunter2333) **Base:** `main` ← **Head:** `feature/email` --- ### 📝 Commits (1) - [`cdd73f2`](https://github.com/dreamhunter2333/cloudflare_temp_email/commit/cdd73f257a1b41c8f649f3bbdaeb71d37210f217) fix: ASN blacklist not working due to missing asnBlacklist field ### 📊 Changes **2 files changed** (+14 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `worker/src/admin_api/ip_blacklist_settings.ts` (+2 -1) 📝 `worker/src/ip_blacklist.ts` (+12 -11) </details> ### 📄 Description ### **User description** - Fix getIpBlacklistSettings() to include asnBlacklist field in return value - Add case-insensitive flag support for regex patterns in ASN matching - Refactor IP blacklist check logic for better code organization 🤖 Generated with [Claude Code](https://claude.com/claude-code) ___ ### **PR Type** Bug fix, Enhancement ___ ### **Description** - Fix missing `asnBlacklist` field in IP blacklist settings. - Add case-insensitive flag support for regex patterns. - Refactor IP blacklist logic for better organization. - Improve handling of IP and ASN blacklist checks. ___ ### **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>ip_blacklist_settings.ts</strong><dd><code>Add `asnBlacklist` field to IP blacklist settings</code>                </dd></summary> <hr> worker/src/admin_api/ip_blacklist_settings.ts <li>Add <code>asnBlacklist</code> field to default settings.<br> <li> Ensure <code>asnBlacklist</code> is included in returned settings. </details> </td> <td><a href="https://github.com/dreamhunter2333/cloudflare_temp_email/pull/756/files#diff-02112f0b34b5d82117c3eebefb434860a3dba70dfd02f607796a0ba2a56c120f">+2/-1</a>      </td> </tr> </table></td></tr><tr><td><strong>Enhancement</strong></td><td><table> <tr> <td> <details> <summary><strong>ip_blacklist.ts</strong><dd><code>Enhance IP and ASN blacklist logic and regex handling</code>        </dd></summary> <hr> worker/src/ip_blacklist.ts <li>Add case-insensitive regex flag for blacklist checks.<br> <li> Include <code>asnBlacklist</code> in settings retrieval logic.<br> <li> Refactor IP blacklist check to handle empty lists.<br> <li> Improve ASN blacklist handling in IP checks. </details> </td> <td><a href="https://github.com/dreamhunter2333/cloudflare_temp_email/pull/756/files#diff-af35186f7408e3585e885c1c86e762daa73ae8ec4b99d6a263c7f330da975e5e">+12/-11</a>  </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>