Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

fix(server): fix bizarre duplicate logs #81

Merged
merged 1 commit into from
Apr 26, 2020
Merged

fix(server): fix bizarre duplicate logs #81

merged 1 commit into from
Apr 26, 2020

Conversation

tamj0rd2
Copy link
Owner

I forgot to return after doing the query validation, so even if the query was found to be invalid it
would still continue trying to serve the request using the same handler

re #50

@codecov
Copy link

codecov bot commented Apr 26, 2020

Codecov Report

Merging #81 into master will increase coverage by 0.28%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   84.34%   84.63%   +0.28%     
==========================================
  Files          36       36              
  Lines         690      690              
  Branches      174      174              
==========================================
+ Hits          582      584       +2     
+ Misses         99       97       -2     
  Partials        9        9              
Impacted Files Coverage Δ
src/commands/serve/server/index.ts 91.17% <100.00%> (-1.48%) ⬇️
src/config/config-builder.ts 100.00% <0.00%> (+12.50%) ⬆️

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 bbc6e06...3aca1c1. Read the comment docs.

I forgot to return after doing the query validation, so even if the query was found to be invalid it
would still continue trying to serve the request using the same handler

re #50
@tamj0rd2 tamj0rd2 merged commit abccc3e into master Apr 26, 2020
@tamj0rd2
Copy link
Owner Author

🎉 This PR is included in version 0.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant