-
Notifications
You must be signed in to change notification settings - Fork 17
Release 7.6.0 #624
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Release 7.6.0 #624
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add a _source property containing the user custom content to match the API.
## What does this PR do? As the title suggests, this **PR** implements the recently added [security:getUserStrategies](https://docs.kuzzle.io/core/2/api/controllers/security/get-user-strategies/) _(@ Kuzzle Backend)_ So it can be used directly, instead of having to use **query** to access it. Closes #611
Closes #593 ## What does this PR do? From now on, when developing with the SDK, you will see warning when using a deprecated action. You can disable this behavior by setting to false the deprecationWarning option on the kuzzle sdk object ### How should this be manually tested? Use a deprecated action and observe the warning(s) ### Other changes Updated doc and added tests
## What does this PR do? Add support for ES suggesters (`suggest` keyword) See also kuzzleio/kuzzle#2019 ### Other changes - reduce the bundle size by ignoring node packages for the web build `230KB => 137KB` (fix #616)
What does this PR do? This PR implements the recently added index:stats (@ Kuzzle Backend v2.10.0) So it can be used directly, instead of having to use query to access it. Todo: Implementation Unit Testing Doc + Code Snippet How should this be manually tested? Launch a Kuzzle stack. Directly use sdk.index.stats() Verify that you get stats that make sense. Closes #608 ℹ️ It seems that Codecov is not taking the latest commit? 🤦
## What does this PR do? Add the `kuzzleStack` property containing the original stacktrace from Kuzzle. It will allow to display only the Kuzzle stacktrace when an error occurs (e.g. in Kourou the user does not care about the SDK stacktrace)
When Kuzzle exposes multiple HTTP routes for the same controller action, the SDK picks either the one with the shortest URL and, if there is a tie in URL lengths, it prefers the GET route to favor content caching. But when multiple routes are available, all with the same size, and none are a GET route, the SDK picks... none of them. Resulting in some actions returning a "URL not found" when trying to use them.
This PR adds the support of cookie when using the websocket protocol. To do that, when Kuzzle cookieAuth option is true, Kuzzle will call the method enableCookieSupport from the given protocol. When called, this method will throw if outside the browser or if cookie are not supported by the protocol, otherwise this will change how the protocol behave. For the HTTP protocol it's simple, when enableCookieSupport is called, the protocol will be changing if the request are made with withCredentials set to true or false depending if he should be able to receive cookies. For the Websocket protocol, this is a bit more complex, when enableCookieSupport is called, the protocol, will be creating a instance of the HTTP Protocol, with the same option (host, port, ssl, ...) as the websocket protocol, after that, when a request auth:login, auth:logout or auth:refreshToken is made, the protocol will use the HTTP Protocol instead of the websocket client to make the request. [ex: auth:login request is made -> websocket closes the connection -> then send the request with the http protocol -> when a response is received it reopens the connection -> then resolve the request]
Codecov Report
@@ Coverage Diff @@
## master #624 +/- ##
==========================================
- Coverage 86.83% 86.30% -0.53%
==========================================
Files 33 34 +1
Lines 1519 1585 +66
Branches 269 279 +10
==========================================
+ Hits 1319 1368 +49
- Misses 145 159 +14
- Partials 55 58 +3
Continue to review full report at Codecov.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
7.6.0 (2021-04-06)
Bug fixes
New features
Enhancements