[Chen Yufeiyang (feiiiiii5)]

Motivation and Context

Valid RFC 2397 data URIs are rejected or silently mis-decoded when the media type contains + (for example image/svg+xml) or when a parameter precedes base64 (for example data:text/plain;charset=utf-8;base64,...).

ImageContent(data_uri="data:image/svg+xml;base64,PHN2Zy8+") raises ContentInitializationError: Invalid data uri format. DataUri.from_data_uri("data:text/plain;charset=utf-8;base64,SGVsbG8=") does not raise, but returns the base64 text as bytes instead of decoding it, because charset=utf-8;base64 is treated as one data-format string. Both are valid data URIs, and the second is a silent data-corruption path.

Description

The parser now uses RFC 6838/RFC 2045 token characters for the media type and parameters, and keeps the optional data format as a token. This accepts image/svg+xml, vendor types such as application/vnd.ms-excel, and ;charset=utf-8;base64 without treating the parameter as part of the base64 marker.

Tests added for the media-type +, a parameter before base64, a vendor media type, and ImageContent construction from an SVG data URI. On the base commit 483ffb4, the four new tests fail; on this branch the related contents/prompt/template/text suites are 1,111 passed. ruff check and ruff format --check are clean.

Contribution Checklist

• The code builds clean without any errors or warnings (imports and targeted unit suites).
• The PR follows the SK contribution guidelines; ruff check and ruff format --check pass on the changed files.
• All targeted unit tests pass, and new regression tests were added.
• No breaking change for existing valid data URIs; quoted-string parameters remain out of scope.

[Copilot]

🟡 Changes recommended

The revised parser rejects valid parameter characters and shorthand URIs that omit the media type.

2 open issues

• Allow omitted MIME type with parameters in data URIs · New
• Use RFC 2045 parameter-token alphabet for data URI parameters · New

What changed in this PR

This PR updates Python’s data URI parser to accept additional media-type characters and correctly separate parameters from the base64 marker.

Changes:

• Expands the parser’s media-type and parameter character classes.
• Adds regression tests for SVG, vendor media types, and charset parameters before base64.
• Tests ImageContent construction from an SVG data URI.

File	Description
python/​tests/​unit/​contents/​test_image_content.py	Adds SVG data URI construction coverage.
python/​tests/​unit/​contents/​test_data_uri.py	Adds media-type and parameter parsing cases.
python/​semantic_kernel/​contents/​utils/​data_uri.py	Revises the data URI parsing expression.

_{Review effort: Balanced}

[Chen Yufeiyang (feiiiiii5)]

Fixed in b0e34c5: the optional media type is now separate from the parameters group. Tests cover data:;charset=utf-8,Hello and its base64 form, alongside explicit-media-type controls. The omitted-media-type cases failed before this fix. The complete contents suite passes 407 tests.

[Chen Yufeiyang (feiiiiii5)]

Fixed in b0e34c5: parameters use the broader token alphabet separately from the media type alphabet. Tests include *, apostrophe, ~, %20, and %3B in parameter values, and assert the parameter remains parsed rather than being silently treated as data_format. All five failed before the fix; the complete contents suite now passes 407 tests.