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
Fixing error that shows we're using X-Pack when we have Basic #3692
Fixing error that shows we're using X-Pack when we have Basic #3692
Conversation
server/controllers/wazuh-api.ts
Outdated
@@ -1060,7 +1060,7 @@ export class WazuhApiCtrl { | |||
|
|||
const disabledRoles = ( await getConfiguration() )['disabled_roles'] || []; | |||
const logoSidebar = ( await getConfiguration() )['customization.logo.sidebar'] || 'icon_blue.png'; | |||
const wazuhSecurity = SecurityObj(context.wazuh.plugins); | |||
const wazuhSecurity = await SecurityObj(context.wazuh.plugins, context); |
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.
nitpick: use the context.wazuh.security
instead of create another security object.
try { | ||
const responseCurl = await context.core.elasticsearch.client.asInternalUser.transport.request( | ||
params | ||
); | ||
} catch (error) { | ||
return !!opendistroSecurityKibana | ||
? new OpendistroFactory(opendistroSecurityKibana) | ||
: new DefaultFactory(); | ||
} |
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.
nitpick: The request to check if X-Pack is enabled should be done only if security
plugin is installed.
…-X-Pack' of github.com:wazuh/wazuh-kibana-app into fix/Fixing-error-with-Basic-that-shows-like-we're-using-X-Pack
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.
review:
Code Review: LGTM
|
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: 🟢
Testing: 🟢 (For Basic & ODFE)
…hows-like-we're-using-X-Pack
SHOULD_TEST=FALSE
Hi guys,
We are fixing a bad behaviour that's shows to Wazuh app like we are using X-Pack when we have Basic Security.
To test it:
Test with X-Pack, Basic and ODFE and be sure that the app is receiving properly which security plugin you are using.
Close this comment