Skip to content

Name the fixed Cloud outbound IP in IP-allowlist adapter texts - #1493

Merged
helpcodeai merged 2 commits into
mainfrom
fix/ip-allowlist-adapter-texts
Nov 3, 2026
Merged

helpcodeai merged 2 commits into
mainfrom
fix/ip-allowlist-adapter-texts

Conversation

@helpcodeai

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #1487. Cloud now calls every API from the fixed address 104.248.242.235 and shows it on the setup page, but several adapter texts still told Cloud users to "ask support for the current egress IP". A Bitrix24 support ticket today was about exactly this: a portal behind an IP restriction. This PR names the address in the adapters whose provider can refuse unknown IPs.

Changes

  • Atlassian, Mercury, Revolut Business, Snowflake, Splunk: replace "ask support for the current egress IP" with "the setup page shows it; on AnythingMCP Cloud it is the fixed IPv4 address 104.248.242.235" (same wording style as eMAG and Korea Law).
  • Snowflake: requiresIpAllowlist: true. A PAT only works for a user under a network policy by default, so the allowlist is a setup step, not a side note.
  • Bitrix24: new Network paragraph (portal IP restriction or self-hosted firewall; "Use proxy" calls always refused by such a list).
  • Google Maps: an IP restriction needs a fixed outbound address; say Cloud has one.
  • T-Bank: name the fixed Cloud address instead of "the Cloud egress address".

Splunk is not flagged requiresIpAllowlist: only Splunk Cloud needs the allow list; Splunk Enterprise does not.

Cloud / self-hosted: texts only. On self-hosted the instructions point to the setup page (which shows EGRESS_IPS when set) and "the address of your server"; the Cloud address is named only as the Cloud case.

Type

  • Documentation (adapter texts) + one adapter flag

Testing

Run in the repository root and packages/backend:

  • node scripts/validate-adapters.mjs --warn: exit 0, 1443 warnings before and after (no new warnings).
  • node scripts/regenerate-catalog.mjs: no diff in catalog.ts.
  • node scripts/adapter-count.mjs --check: OK (695 adapters).
  • node scripts/generate-llms-txt.mjs: no diff.
  • npx jest --maxWorkers=4 on the static specs of atlassian, bitrix24, google-maps, mercury, revolut-business, snowflake, tbank plus catalog.spec.ts and instruction-facts.spec.ts: 9 suites passed (14836 tests). adapters.service.spec.ts and catalog-summary.spec.ts could not run locally (shared node_modules without @types/picomatch, unrelated to this change); left to CI.

Related Issues

Relates to #1487

Atlassian, Mercury, Revolut Business, Snowflake and Splunk told Cloud users
to ask support for the egress IP, which is now fixed and shown on the setup
page. Name 104.248.242.235 instead, as eMAG and Korea Law already do. Mark
Snowflake requiresIpAllowlist (a PAT needs a network policy by default), add
a network note to Bitrix24 (portals behind an IP restriction or firewall),
and update the Google Maps and T-Bank address hints.
@helpcodeai
helpcodeai requested a review from D3nisty as a code owner November 3, 2026 16:15
@helpcodeai
helpcodeai enabled auto-merge (squash) November 3, 2026 16:15
@helpcodeai
helpcodeai merged commit 7c2f749 into main Nov 3, 2026
14 checks passed
@helpcodeai
helpcodeai deleted the fix/ip-allowlist-adapter-texts branch November 3, 2026 16:27
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant