Skip to content

fix/align_doc_on_various_regions#2994

Merged
alexane-bougeardbebin-sekoia merged 6 commits into
mainfrom
fix/align_https_doc_on_various_regions
May 20, 2026
Merged

fix/align_doc_on_various_regions#2994
alexane-bougeardbebin-sekoia merged 6 commits into
mainfrom
fix/align_https_doc_on_various_regions

Conversation

@clement-burtscher-sekoia
Copy link
Copy Markdown
Contributor

@clement-burtscher-sekoia clement-burtscher-sekoia commented May 19, 2026

Description

Changelog

Added

  • Include information and examples for all supported regions (FRA1, FRA2, MCO1, UAE1, USA1, SGP1)
  • Clarify URL scheme differences: FRA1 legacy vs API-prefixed scheme for other regions
  • Label existing examples as FRA1-specific and warn users to adapt code samples for other regions

Fixed

  • Correct field name typo: intakey_keyintake_key in structured content section
  • Improve wording: "query string" (was "querystring"), "Sekoia.io's" capitalization

@clement-burtscher-sekoia clement-burtscher-sekoia requested review from a team and Copilot May 19, 2026 12:55
@clement-burtscher-sekoia clement-burtscher-sekoia added bug Something isn't working documentation Improvements or additions to documentation labels May 19, 2026
@clement-burtscher-sekoia clement-burtscher-sekoia changed the title Rework https doc to be multi-regions oriented fix/align_https_doc_on_various_regions May 19, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the HTTPS ingestion formatting documentation to be region-aware, adding guidance for selecting the correct HTTP intake endpoint per Sekoia.io region and clarifying URL scheme differences between FRA1 and other regions.

Changes:

  • Added a “Select the intake endpoint for your region” section with a region→endpoint table and derivation guidance for ingestion paths.
  • Updated wording/typography in multiple places (e.g., “query string”, possessive casing).
  • Fixed a structured payload field name typo: intakey_keyintake_key.
Comments suppressed due to low confidence (1)

docs/integration/ingestion_methods/https/format.md:32

  • The suggested base URL for non-FRA1 regions is https://intake.<region>.io/api/v1/intake-http, but other docs use https://intake.<region>.sekoia.io/api/v1/intake-http (often with a trailing /). Please update this derivation rule to match the canonical regional intake base URL format to avoid generating broken ingestion URLs.
- FRA1 base URL: `https://intake.sekoia.io`
- Other regions base URL: `https://intake.<region>.io/api/v1/intake-http`


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/integration/ingestion_methods/https/format.md Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 19, 2026

Newest code from clement-burtscher-sekoia has been published to preview environment

🚀 Latest deployment was built on 2026-05-20 08:33:41 (d64d0ca2a6bf4b129de81497f1b7bccc692f7dda).

@clement-burtscher-sekoia clement-burtscher-sekoia force-pushed the fix/align_https_doc_on_various_regions branch from 5662d49 to d9d7517 Compare May 19, 2026 13:10
@clement-burtscher-sekoia clement-burtscher-sekoia changed the title fix/align_https_doc_on_various_regions fix/align_doc_on_various_regions May 19, 2026
Comment thread docs/integration/ingestion_methods/https/format.md
Comment thread docs/integration/ingestion_methods/https/format.md Outdated
Comment thread _shared_content/automate/playbooks-on-premises.md Outdated
Comment thread _shared_content/automate/playbooks-on-premises.md
Comment thread _shared_content/automate/playbooks-on-premises.md Outdated
Comment thread _shared_content/automate/playbooks-on-premises.md Outdated
@clement-burtscher-sekoia clement-burtscher-sekoia force-pushed the fix/align_https_doc_on_various_regions branch from 6c51e9a to 47dc972 Compare May 20, 2026 08:10
@alexane-bougeardbebin-sekoia alexane-bougeardbebin-sekoia merged commit 5ff777a into main May 20, 2026
3 checks passed
@alexane-bougeardbebin-sekoia alexane-bougeardbebin-sekoia deleted the fix/align_https_doc_on_various_regions branch May 20, 2026 08:39
@github-actions
Copy link
Copy Markdown

Pull request was merged, preview was removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants