Skip to content

examples: fix chat_with_image using a dead image URL#257

Merged
Nelson-PROIA merged 2 commits into
mainfrom
nelsonproia/fix-example-image-url
Jul 23, 2026
Merged

examples: fix chat_with_image using a dead image URL#257
Nelson-PROIA merged 2 commits into
mainfrom
nelsonproia/fix-example-image-url

Conversation

@Nelson-PROIA

Copy link
Copy Markdown
Contributor

async_chat_with_image_no_streaming.ts pointed at a deleted asset, so the API returned 400 invalid_request_file. Use a live image. Verified the example runs against the API.

The image URL pointed at a deleted asset, so the API returned 400 invalid_request_file. Use a live image.
@andreaonofrei01
andreaonofrei01 self-requested a review July 20, 2026 09:06

@andreaonofrei01 andreaonofrei01 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I have a qq about the new image URL https://mistral.ai/_astro/ai-app_Z2q9iqE.webp?dpl=6a57bb9ad483ec680851599b. from my quick search it's a content-hashed Astro build artifact. Assuming that's what it is: ?dpl=... appears to be a per-deploy identifier.
my question is wouldn't the URL change whenever mistral.ai is redeployed and eventually 404 ? which would reintroduce the same problem this PR is fixing. I haven't confirmed this, so just wanted to check whether the URL is actually stable.

Replace the mistral.ai _astro build asset (per-deploy content hash, would 404 on redeploy) with the Mistral logo pinned to a commit SHA so it can never rot. Addresses review feedback.
@Nelson-PROIA
Nelson-PROIA merged commit 61f6e88 into main Jul 23, 2026
4 checks passed
@Nelson-PROIA
Nelson-PROIA deleted the nelsonproia/fix-example-image-url branch July 23, 2026 09:56
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