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

enhance: add remediation supported features bubble #624

Merged
merged 12 commits into from
Mar 20, 2025

Conversation

LaurenceJJones
Copy link
Contributor

To help user see from a glance what feature the remediation supports

Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-624.d1to60jd2gb6y6.amplifyapp.com

@AlteredCoder
Copy link
Contributor

Good idea for the bubble! The only weird thing is the "Mode" bubble, which should not be a boolean (because it can live, stream or both). It leads to "Unsupported" mode for many remediation components.
Furthermore, maybe we should call it "Decisions Mode" or something else that only "Mode"?

@LaurenceJJones LaurenceJJones marked this pull request as draft November 5, 2024 10:26
@LaurenceJJones
Copy link
Contributor Author

converting to draft till I have time to solve merge conflicts

@rr404
Copy link
Contributor

rr404 commented Feb 10, 2025

should we add a page for traefik ? it is appsec able

@rr404 rr404 force-pushed the remediation_support_features branch from b762799 to b8f0bd5 Compare March 18, 2025 13:17
@rr404
Copy link
Contributor

rr404 commented Mar 18, 2025

Good idea for the bubble! The only weird thing is the "Mode" bubble, which should not be a boolean (because it can live, stream or both). It leads to "Unsupported" mode for many remediation components. Furthermore, maybe we should call it "Decisions Mode" or something else that only "Mode"?

Can we assume that a bouncer with only one mode will have only Live ? so we just change the labels and not the logic of laurence's code (otherwise we'll have to passed something more than just a boolà)

@LaurenceJJones
Copy link
Contributor Author

Can we assume that a bouncer with only one mode will have only Live

Most bouncers without a mode are stream typically.

<RemediationSupportBadge title='Metrics' description='Can send detailed metrics to LAPI' supported={Metrics} />
<RemediationSupportBadge title='MTLS' description='Can do mutual TLS authentication to LAPI' supported={MTLS} />
<RemediationSupportBadge title='Prometheus' description='Can expose metrics to Prometheus' supported={Prometheus} />
<RemediationSupportBadge title='Mode' description='Can be configured in different modes, typically live or stream' support={modeSupport} />
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the description doesnt really match the present values anymore.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

true, I was just testing to see what looked like. but we need to do a slight improve on the Mode bubble

Copy link
Contributor

@rr404 rr404 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for v0
We'll make updates immediatly for a few bouncers after this merge

@LaurenceJJones LaurenceJJones marked this pull request as ready for review March 20, 2025 14:02
@rr404 rr404 merged commit b18d0ac into crowdsecurity:main Mar 20, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants