Skip to content
This repository has been archived by the owner on Nov 25, 2024. It is now read-only.

Add CORP header to /download and /thumbnail #3299

Merged
merged 1 commit into from
Jan 10, 2024
Merged

Conversation

S7evinK
Copy link
Contributor

@S7evinK S7evinK commented Jan 10, 2024

@S7evinK S7evinK added the spec-compliance Fix something that doesn't comply with the specs label Jan 10, 2024
@S7evinK S7evinK requested a review from a team as a code owner January 10, 2024 07:58
Copy link

codecov bot commented Jan 10, 2024

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (57646d5) 65.34% compared to head (888f63a) 65.38%.

Files Patch % Lines
mediaapi/routing/routing.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3299      +/-   ##
==========================================
+ Coverage   65.34%   65.38%   +0.03%     
==========================================
  Files         509      509              
  Lines       57481    57482       +1     
==========================================
+ Hits        37563    37586      +23     
+ Misses      16022    16003      -19     
+ Partials     3896     3893       -3     
Flag Coverage Δ
unittests 50.00% <0.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@S7evinK S7evinK merged commit e342420 into main Jan 10, 2024
17 of 20 checks passed
@S7evinK S7evinK deleted the s7evink/media-corp-header branch January 10, 2024 08:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
spec-compliance Fix something that doesn't comply with the specs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant