Skip to content
This repository was archived by the owner on Jun 30, 2023. It is now read-only.

fix array path parameter deserialization#137

Merged
tangiel merged 1 commit intomasterfrom
array-path-param
Feb 17, 2018
Merged

fix array path parameter deserialization#137
tangiel merged 1 commit intomasterfrom
array-path-param

Conversation

@tangiel
Copy link
Contributor

@tangiel tangiel commented Feb 17, 2018

This change fixes path parameter deserialization when the parameter is
an array-like structure. Previously, this only worked for query
parameters, and a string literal was injected as an array for path
parameters.

This change fixes path parameter deserialization when the parameter is
an array-like structure. Previously, this only worked for query
parameters, and a string literal was injected as an array for path
parameters.
@tangiel tangiel requested a review from inklesspen February 17, 2018 00:10
@codecov-io
Copy link

codecov-io commented Feb 17, 2018

Codecov Report

Merging #137 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #137      +/-   ##
============================================
+ Coverage     79.91%   79.93%   +0.02%     
- Complexity     1668     1670       +2     
============================================
  Files           156      156              
  Lines          5580     5586       +6     
  Branches        719      721       +2     
============================================
+ Hits           4459     4465       +6     
  Misses          843      843              
  Partials        278      278
Impacted Files Coverage Δ Complexity Δ
...ver/spi/request/RestServletRequestParamReader.java 93.33% <100%> (+0.74%) 23 <0> (+2) ⬆️

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 968f420...6af1961. Read the comment docs.

@tangiel tangiel merged commit 8378711 into master Feb 17, 2018
@tangiel tangiel deleted the array-path-param branch February 17, 2018 00:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants