Skip to content

airframe-http: Add CORS filter and filter-instance support#846

Merged
xerial merged 2 commits intowvlet:masterfrom
xerial:cors
Dec 7, 2019
Merged

airframe-http: Add CORS filter and filter-instance support#846
xerial merged 2 commits intowvlet:masterfrom
xerial:cors

Conversation

@xerial
Copy link
Copy Markdown
Member

@xerial xerial commented Dec 7, 2019

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 7, 2019

Codecov Report

Merging #846 into master will decrease coverage by <.01%.
The diff coverage is 68.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #846      +/-   ##
==========================================
- Coverage   81.03%   81.03%   -0.01%     
==========================================
  Files         227      228       +1     
  Lines        8994     9004      +10     
  Branches      639      651      +12     
==========================================
+ Hits         7288     7296       +8     
- Misses       1706     1708       +2
Impacted Files Coverage Δ
...t/airframe/http/router/HttpRequestDispatcher.scala 90.9% <100%> (+0.28%) ⬆️
...vlet/airframe/http/finagle/filter/CorsFilter.scala 100% <100%> (ø)
...a/wvlet/airframe/http/finagle/FinagleBackend.scala 72% <100%> (+3.81%) ⬆️
...tp/src/main/scala/wvlet/airframe/http/Router.scala 64.44% <50%> (-1.41%) ⬇️

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 d342530...fb3bcf1. Read the comment docs.

@xerial xerial mentioned this pull request Dec 7, 2019
76 tasks
@xerial xerial merged commit 58245b2 into wvlet:master Dec 7, 2019
@xerial xerial deleted the cors branch December 7, 2019 02:33
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.

1 participant