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

upgrade coraza to v3.0.3 #84

Closed
wants to merge 1 commit into from
Closed

upgrade coraza to v3.0.3 #84

wants to merge 1 commit into from

Conversation

jptosso
Copy link
Member

@jptosso jptosso commented Aug 21, 2023

No description provided.

@jcchavezs
Copy link
Member

@M4tteoP e2e seems stuck here. Could you please migrate to new e2e?

@M4tteoP
Copy link
Member

M4tteoP commented Aug 21, 2023

Tried to give it a go here: #85. Still a WIP, it is currently failing.
What I notice is that the e2e test started failing from the latest commit of #72, but I can't find anything obvious that originated this problem. cc @sts

@zc-devs
Copy link
Contributor

zc-devs commented Aug 21, 2023

Perhaps, it's because of no_response_check: true. If yes, you can pick up fixes from here.

@jcchavezs
Copy link
Member

jcchavezs commented Aug 21, 2023 via email

@zc-devs
Copy link
Contributor

zc-devs commented Aug 22, 2023

Up for a PR?

Unfortunately, no. I'm quite busy now.

@fzipi
Copy link
Member

fzipi commented Jan 2, 2024

@zc-devs Do you think it is a better time now to provide a PR?

@zc-devs
Copy link
Contributor

zc-devs commented Jan 2, 2024

Sorry, I've changed my mind towards Kubernetes. I'm running CrowdSec now and probably will deploy open-appsec. So, I've been not using Coraza SPOA since then.

Anyway, it should be this commit. Or just revert this setting.

@M4tteoP M4tteoP mentioned this pull request Jan 2, 2024
@fionera fionera mentioned this pull request Mar 5, 2024
5 tasks
@jptosso jptosso closed this in #103 Nov 7, 2024
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.

5 participants