Skip to content

add *.bingx.pro *.bingxzone.com to whitelist#1745

Open
deboyblog wants to merge 3 commits into
phantom:masterfrom
deboyblog:master
Open

add *.bingx.pro *.bingxzone.com to whitelist#1745
deboyblog wants to merge 3 commits into
phantom:masterfrom
deboyblog:master

Conversation

@deboyblog
Copy link
Copy Markdown

@deboyblog deboyblog commented Dec 10, 2025

I admin the website bingx.pro and bingxzone.com.
Both of these site are backup domain for some counrty.
we face phantom browser alerts our site is scam.

Please ensure the white list is updated to include those two domains.
Thank you

Summary by CodeRabbit

  • Chores
    • Expanded the allowlist to recognize additional domain patterns, improving compatibility with extended service infrastructure while maintaining all existing allowlisted entries.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 12, 2026

📝 Walkthrough

Walkthrough

The allowlist configuration extends to include four new BingX subdomain patterns (*.bingx.pro, *.bingx.io, *.bingxzone.com, *.bingxtech.com) while retaining the existing nftplus.io entry.

Changes

BingX subdomain allowlist entries

Layer / File(s) Summary
BingX subdomain allowlist entries
whitelist.yaml
Four BingX domain patterns are added to the allowlist configuration adjacent to the existing nftplus.io entry.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title partially addresses the changeset but omits two domains (*.bingx.io and *.bingxtech.com) that were actually added, making it incomplete and somewhat misleading. Update the title to include all four domains added: 'add *.bingx.pro *.bingx.io *.bingxzone.com *.bingxtech.com to whitelist' or summarize as 'add bingx-related domains to whitelist'.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@whitelist.yaml`:
- Around line 34-37: The whitelist.yaml currently adds wildcard entries
(*.bingx.pro, *.bingx.io, *.bingxzone.com, *.bingxtech.com) but three domains
lack confirmed ownership; verify ownership of bingx.io, bingxzone.com, and
bingxtech.com using WHOIS, SSL cert subject/issuer, authoritative DNS records or
direct confirmation from BingX, then document those verification results in the
PR; if a domain cannot be confirmed, remove it from whitelist.yaml (or replace
the wildcard with only explicitly verified subdomains) and update the PR
description to match the actual changes (ensure only the originally requested
domains are added or provide evidence for any extras).
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8e2e49b2-d03d-49c6-9e1e-2f4f4e6c5c4c

📥 Commits

Reviewing files that changed from the base of the PR and between f0e8ff9 and 6191b7a.

📒 Files selected for processing (1)
  • whitelist.yaml

Comment thread whitelist.yaml
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