Skip to content

airframe-http: Cross-platform Request and Response classes#982

Merged
xerial merged 15 commits intowvlet:masterfrom
xerial:js-client-adapter
Mar 15, 2020
Merged

airframe-http: Cross-platform Request and Response classes#982
xerial merged 15 commits intowvlet:masterfrom
xerial:js-client-adapter

Conversation

@xerial
Copy link
Copy Markdown
Member

@xerial xerial commented Mar 15, 2020

Fixes #827

@xerial xerial mentioned this pull request Mar 15, 2020
76 tasks
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 15, 2020

Codecov Report

Merging #982 into master will decrease coverage by 0.98%.
The diff coverage is 43.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #982      +/-   ##
==========================================
- Coverage    82.1%   81.12%   -0.99%     
==========================================
  Files         258      264       +6     
  Lines        9653     9867     +214     
  Branches      654      662       +8     
==========================================
+ Hits         7926     8005      +79     
- Misses       1727     1862     +135
Impacted Files Coverage Δ
.../main/scala/wvlet/airframe/http/router/Route.scala 68.75% <ø> (ø) ⬆️
...cala/wvlet/airframe/http/router/RouteMatcher.scala 87.14% <ø> (ø) ⬆️
...la/wvlet/airframe/http/finagle/FinagleClient.scala 91.3% <ø> (ø) ⬆️
...airframe/http/codegen/client/ScalaHttpClient.scala 100% <ø> (ø) ⬆️
...ala/wvlet/airframe/http/codegen/HttpClientIR.scala 96.66% <ø> (ø) ⬆️
...rc/main/scala/wvlet/airframe/http/HttpHeader.scala 0% <0%> (ø)
...rc/main/scala/wvlet/airframe/http/HttpMethod.scala 0% <0%> (ø)
...wvlet/airframe/http/router/HttpRequestMapper.scala 96.77% <100%> (ø) ⬆️
...in/scala/wvlet/airframe/http/finagle/package.scala 95.55% <100%> (+0.68%) ⬆️
.../scala/wvlet/airframe/http/HttpMultiMapCodec.scala 20% <20%> (ø)
... and 10 more

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 58ef3ff...cab4288. Read the comment docs.

@xerial xerial changed the title airframe-http: Use HttpMultiMap for https headers and query parameters airframe-http: Cross-platform Request and Response classes Mar 15, 2020
@xerial xerial merged commit f55d438 into wvlet:master Mar 15, 2020
@xerial xerial deleted the js-client-adapter branch March 15, 2020 09:16
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.

airframe-http: http backend independent request/response objects

1 participant