fix: allow httpStorage `allowAllDomains` config param to be used.

unjs / ipx

🖼️ High performance, secure and easy-to-use image optimizer.

MIT License

1.2k stars 59 forks source link

fix: allow httpStorage `allowAllDomains` config param to be used. #189

Closed Jarrku closed 8 months ago

Jarrku commented 8 months ago

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

The allowAllDomains option is documented in the TS types, but currently it not used. Added a change so when the value is passed in, it's also used.

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

codecov[bot] commented 8 months ago

Codecov Report

Merging #189 (61a0d2c) into main (4828679) will increase coverage by 0.04%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #189      +/-   ##
==========================================
+ Coverage   54.01%   54.05%   +0.04%     
==========================================
  Files          11       11              
  Lines        1083     1084       +1     
  Branches       45       45              
==========================================
+ Hits          585      586       +1     
  Misses        498      498

Files	Coverage Δ
src/storage/http.ts	`58.33% <100.00%> (+0.38%)`	:arrow_up: