Skip to content

Conversation

@davea
Copy link
Member

@davea davea commented Dec 18, 2025

Add support for decoding base64-encoded data URIs when processing media URLs. The method now checks for the ;base64 marker and decodes the data appropriately.

For FD-6433

Add support for decoding base64-encoded data URIs when processing
media URLs. The method now checks for the ;base64 marker and decodes
the data appropriately.

For FD-6433
@davea davea force-pushed the handle-base64-data-uris branch from ce8f573 to b5657b5 Compare December 18, 2025 13:10
@davea davea requested a review from MorayMySoc December 18, 2025 13:10
@davea davea marked this pull request as ready for review December 18, 2025 13:10
@codecov
Copy link

codecov bot commented Dec 18, 2025

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 82.65%. Comparing base (08bfe41) to head (b5657b5).

Files with missing lines Patch % Lines
perllib/Open311.pm 83.33% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5785      +/-   ##
==========================================
+ Coverage   82.64%   82.65%   +0.01%     
==========================================
  Files         457      457              
  Lines       35716    35721       +5     
  Branches     5822     5824       +2     
==========================================
+ Hits        29516    29524       +8     
  Misses       4501     4501              
+ Partials     1699     1696       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

3 participants