Avoid force check updates when user logins #6375
Merged
Conversation
This file contains 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
|
|
|
chantal-kelm
approved these changes
Jan 25, 2024
TestAfter log in
Server APIs
Check updates button
Check updates button tooltip
|
Tostti
deleted the
enhancement/6308-avoid-force-check-updates-when-user-logins
branch
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.
Description
This PR prevents checking for updates with the force_query parameter set to true upon user login. Consequently, the Wazuh API will now return available updates without querying the CTI Service.
Issues Resolved
#6308
Evidence
query_api: Determines whether the Check Updates plugin retrieves data from the Wazuh API or from a saved object.force_query: Whenquery_apiis set to true, it determines whether the Wazuh API internally obtains the data or fetches it from the CTI Service.After user logins
Server APIs page
Check updates button
Update Check updates button tooltip
Test
/api/wazuh-check-updates/updates?query_api=true&force_query=falseIndexer/dashboard management -> Server APIsand check de request:/api/wazuh-check-updates/updates?query_api=false&force_query=falseCheck updatesbutton and check the request:/api/wazuh-check-updates/updates?query_api=true&force_query=trueCheck List
yarn test:jest