Panic on wildcard in Cors builder's allowed_origin()

[eupn]

PR Type

Feature

PR Checklist

Check your PR fulfills the following:

• Tests for the changes have been added / updated.
• Documentation comments have been added / updated.
• A changelog entry has been made for the appropriate packages.
• Format code with the latest stable rustfmt

Overview

Implements #112.

[robjtede]

thanks for picking this up :)

just some initial thoughts, will sanity check running it later

[codecov]

Codecov Report

Merging #114 into master will increase coverage by 0.43%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
+ Coverage   79.45%   79.89%   +0.43%     
==========================================
  Files          29       29              
  Lines        2098     2104       +6     
==========================================
+ Hits         1667     1681      +14     
+ Misses        431      423       -8     

Impacted Files	Coverage Δ
actix-cors/tests/tests.rs	99.20% <66.66%> (-0.80%)	⬇️
actix-cors/src/builder.rs	82.70% <100.00%> (+0.26%)	⬆️
...headers/www_authenticate/challenge/bearer/tests.rs	100.00% <0.00%> (ø)
actix-redis/src/session.rs	81.60% <0.00%> (+0.76%)	⬆️
actix-session/src/cookie.rs	89.16% <0.00%> (+0.98%)	⬆️
actix-session/src/lib.rs	89.13% <0.00%> (+1.08%)	⬆️
actix-cors/src/inner.rs	80.90% <0.00%> (+1.81%)	⬆️
actix-cors/src/middleware.rs	72.36% <0.00%> (+2.63%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 134e43a...9d00d19. Read the comment docs.