Skip to content
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

fix: recommend #467

Merged
merged 2 commits into from
Dec 17, 2024
Merged

fix: recommend #467

merged 2 commits into from
Dec 17, 2024

Conversation

jokestax
Copy link
Contributor

@jokestax jokestax commented Nov 9, 2024

Description

check if the auth hash is empty which otherwise throws a panic error

Fixes

Link to issue

Testing

Build the binary locally,and run "./kubearmor-client recommend"

@jokestax
Copy link
Contributor Author

@DelusionalOptimist @daemon1024 please review :)

Signed-off-by: mrrishi <[email protected]>
@jokestax jokestax changed the title fix: recommend fix: recommend and probe command Nov 15, 2024
@jokestax jokestax changed the title fix: recommend and probe command fix: recommend Nov 15, 2024
@Aryan-sharma11
Copy link
Member

Hi @jokestax, kindly rebase this PR to resolve the CI issue.

@jokestax
Copy link
Contributor Author

@Aryan-sharma11 done ✅

@Aryan-sharma11 Aryan-sharma11 merged commit 9ad1b99 into kubearmor:main Dec 17, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants