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
Erasing windows update option in wrong cases #3569
Erasing windows update option in wrong cases #3569
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CR: LGTM ✔️
Testing: LGTM ✔️
e4af80d
to
76ba745
Compare
@@ -1906,7 +1906,7 @@ export class WazuhReportingCtrl { | |||
] | |||
} | |||
}, | |||
{ | |||
agentOs === 'windows' && { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a bad implementation, is not a good practice using a ternary in an array definition
please use a different approach, maybe inserting after the declaration of the array with a conditional
also, check if this endpoint is used for another SO but windows
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I change it using your suggestions. Thanks you!
…tps://github.com/wazuh/wazuh-kibana-app into fix/3566-windows-update-in-linux-agent-inventory
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Hi guys,
We disabled Windows Update part of Inventory PDF in agents with an operative system different from Windows.
To test it, go to Agents, click an agent without Windows, go to Inventory Section and generate a PDF. Then go to the end of that and see if Windows Update section appers.
If you can, try as well with a Windows agent to check that in this case, this section appears.
Closes #3566