Skip to content

Conversation

Aschen
Copy link
Contributor

@Aschen Aschen commented Jul 16, 2019

What does this PR do?

Emit a queryError when receiving a response from a malformed request

Fix #357

@codecov
Copy link

codecov bot commented Jul 16, 2019

Codecov Report

Merging #423 into 6-dev will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           6-dev     #423      +/-   ##
=========================================
+ Coverage   96.3%   96.31%   +<.01%     
=========================================
  Files         32       32              
  Lines       1517     1518       +1     
=========================================
+ Hits        1461     1462       +1     
  Misses        56       56
Impacted Files Coverage Δ
src/protocols/websocket.js 100% <100%> (ø) ⬆️

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 079d964...a1b3cc0. Read the comment docs.

@codecov
Copy link

codecov bot commented Jul 16, 2019

Codecov Report

Merging #423 into 6-dev will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##            6-dev     #423      +/-   ##
==========================================
+ Coverage   96.41%   96.42%   +<.01%     
==========================================
  Files          32       32              
  Lines        1535     1538       +3     
==========================================
+ Hits         1480     1483       +3     
  Misses         55       55
Impacted Files Coverage Δ
src/protocols/websocket.js 100% <100%> (ø) ⬆️

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 4f2ab1a...6408fe4. Read the comment docs.

@Aschen Aschen added the wip label Jul 16, 2019
@Aschen Aschen removed the wip label Jul 18, 2019
Co-Authored-By: Sébastien Cottinet <[email protected]>
@Aschen Aschen changed the title Add discarded response event Emit queryError event on malformed request Jul 24, 2019
@scottinet scottinet merged commit fdd6e83 into 6-dev Jul 25, 2019
@scottinet scottinet deleted the 357-add-discarded-response-event branch July 25, 2019 09:53
@scottinet scottinet mentioned this pull request Jul 31, 2019
scottinet added a commit that referenced this pull request Jul 31, 2019
# [6.2.0](https://github.com/kuzzleio/sdk-javascript/releases/tag/6.2.0) (2019-07-31)


#### Bug fixes

- [ [#428](#428) ] Properly handle boolean flags in HTTP querystrings   ([scottinet](https://github.com/scottinet))
- [ [#427](#427) ] Solve promise+event+memory leaks when the network fails   ([scottinet](https://github.com/scottinet))
- [ [#424](#424) ] Prevent pending request leak when disconnect the SDK   ([Aschen](https://github.com/Aschen))
- [ [#422](#422) ] Fix bug when decoding JWT in browser   ([Aschen](https://github.com/Aschen))
- [ [#420](#420) ] Fix http protocol unresolved promise on connection error   ([Aschen](https://github.com/Aschen))

#### New features

- [ [#419](#419) ] Add bulk:write and bulk:mWrite   ([Aschen](https://github.com/Aschen))

#### Enhancements

- [ [#421](#421) ] Get api routes from server:publicApi   ([Aschen](https://github.com/Aschen))
- [ [#423](#423) ] Emit queryError event on malformed request   ([Aschen](https://github.com/Aschen))
- [ [#417](#417) ] Security controller documentation   ([benoitvidis](https://github.com/benoitvidis))
---
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants