Skip to content

Lowercase Vidyard thumbnail URLs in package docs#18032

Merged
theletterf merged 3 commits intomainfrom
fix/lowercase-vidyard-thumbnails
Mar 25, 2026
Merged

Lowercase Vidyard thumbnail URLs in package docs#18032
theletterf merged 3 commits intomainfrom
fix/lowercase-vidyard-thumbnails

Conversation

@theletterf
Copy link
Copy Markdown
Member

Summary

  • Lowercases Vidyard video thumbnail image URLs in 4 packages: crowdstrike, m365_defender, microsoft_defender_endpoint, sentinel_one
  • Updates both docs/ and _dev/build/docs/ files

Why

docs-builder rejects image filenames with uppercase characters. When integration-docs downloads these thumbnails from play.vidyard.com, the mixed-case filenames (e.g. VKKWSpg4sDEk1DBXATkyEP.jpg) cause the build to fail.

Vidyard serves thumbnails case-insensitively, so lowercasing the URLs is safe.

See: https://github.com/elastic/integration-docs/actions/runs/23537107911/job/68515036838?pr=1152

🤖 Generated with Claude Code

docs-builder rejects image filenames with uppercase characters.
Lowercase the play.vidyard.com thumbnail URLs so the filenames
integration-docs downloads are valid.

Affects: crowdstrike, m365_defender, microsoft_defender_endpoint,
sentinel_one

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@theletterf theletterf requested a review from a team as a code owner March 25, 2026 10:50
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 25, 2026

✅ Vale Linting Results

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@theletterf theletterf self-assigned this Mar 25, 2026
@theletterf theletterf added the bugfix Pull request that fixes a bug issue label Mar 25, 2026
Run elastic-package changelog add + build for each affected package:
- crowdstrike 3.12.0 → 3.12.1
- m365_defender 5.12.0 → 5.12.1
- microsoft_defender_endpoint 4.5.0 → 4.5.1
- sentinel_one 2.5.0 → 2.5.1

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@theletterf theletterf enabled auto-merge (squash) March 25, 2026 11:11
Copy link
Copy Markdown
Contributor

@kcreddy kcreddy left a comment

Choose a reason for hiding this comment

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

lgtm

@theletterf theletterf disabled auto-merge March 25, 2026 11:18
@theletterf theletterf enabled auto-merge (squash) March 25, 2026 11:18
@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

cc @theletterf

@theletterf theletterf merged commit 6be35e4 into main Mar 25, 2026
13 checks passed
@theletterf theletterf deleted the fix/lowercase-vidyard-thumbnails branch March 25, 2026 11:51
@andrewkroh andrewkroh added Integration:crowdstrike CrowdStrike Integration:m365_defender Microsoft Defender XDR Integration:microsoft_defender_endpoint Microsoft Defender for Endpoint Integration:sentinel_one SentinelOne documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. labels Mar 25, 2026
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

Package crowdstrike - 3.12.1 containing this change is available at https://epr.elastic.co/package/crowdstrike/3.12.1/

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

Package m365_defender - 5.12.1 containing this change is available at https://epr.elastic.co/package/m365_defender/5.12.1/

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

Package microsoft_defender_endpoint - 4.5.1 containing this change is available at https://epr.elastic.co/package/microsoft_defender_endpoint/4.5.1/

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

Package sentinel_one - 2.5.1 containing this change is available at https://epr.elastic.co/package/sentinel_one/2.5.1/

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

Labels

bugfix Pull request that fixes a bug issue documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:crowdstrike CrowdStrike Integration:m365_defender Microsoft Defender XDR Integration:microsoft_defender_endpoint Microsoft Defender for Endpoint Integration:sentinel_one SentinelOne

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants