Skip to content

Conversation

@Shiranuit
Copy link
Contributor

What does this PR do ?

Emit a reconnectionError when there is no authenticator set.

@codecov
Copy link

codecov bot commented Sep 20, 2021

Codecov Report

Merging #664 (12e8c52) into 7-dev (1900bf0) will increase coverage by 0.26%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##            7-dev     #664      +/-   ##
==========================================
+ Coverage   85.68%   85.95%   +0.26%     
==========================================
  Files          36       36              
  Lines        1712     1716       +4     
  Branches      311      314       +3     
==========================================
+ Hits         1467     1475       +8     
+ Misses        183      179       -4     
  Partials       62       62              
Impacted Files Coverage Δ
src/Kuzzle.ts 85.79% <100.00%> (+0.04%) ⬆️
src/protocols/Http.ts 84.00% <0.00%> (+0.86%) ⬆️
src/controllers/Document.ts 69.00% <0.00%> (+3.00%) ⬆️

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 1900bf0...12e8c52. Read the comment docs.

Co-authored-by: Adrien Maret <amaret93@gmail.com>
@Aschen Aschen merged commit f41f979 into 7-dev Sep 21, 2021
@Aschen Aschen deleted the emit-reconnection-error branch September 21, 2021 14:39
This was referenced Sep 21, 2021
Aschen added a commit that referenced this pull request Sep 21, 2021
# [7.7.6](https://github.com/kuzzleio/sdk-javascript/releases/tag/7.7.6) (2021-09-21)


#### Enhancements


- [ [#663](#663) ] Convert Security Controller to Typescript   ([inform880](https://github.com/inform880))

#### Bug fixes

- [ [#664](#664) ] Fix reconnectionError not emittend when authenticator is missing   ([Shiranuit](https://github.com/Shiranuit))
---
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