Comments (6)
I would leave it as it is, I don't think this is going to pop up any more after fixing the documentation and would rather not introduce a breaking change for this. In the end there are many examples of applications which wouldn't add a unit into a settings name.
from coraza-spoa.
I think we check if the problem is still there after upgrading coraza to latest cc @jptosso
from coraza-spoa.
I think this is related to the cache service used to share transactions between requests and response. The GC should eventually clean this. I will take a look.
If the transaction is processed for request and response it gets automatically cleaned. Otherwise you must wait for ttl
from coraza-spoa.
The comment in the default configuration was incorrect. The transaction_ttl
setting is in seconds rather than milliseconds. In the user provided configuration, the transactions are cached for 60000 seconds.
This setting should match the sum of timeout connect
+ timeout queue
+ timeout server
) in haproxy.
The following graph illustrates the memory usage when the ttl is set to 10 seconds.
(from slack)
from coraza-spoa.
from coraza-spoa.
Is the name of the setting transaction_ttl something we control? If so, I
think we should change it into something like transaction_ttl_in_secs or
something like that and make it explicit.On Sat, 31 Dec 2022, 09:56 Stefan Schlesinger, @.***>
wrote:—
Reply to this email directly, view it on GitHub
#40 (comment), or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYAUTVPBDX7SMFZRHGFLWP7YNPANCNFSM6AAAAAASLD77OI
.
You are receiving this because you commented.Message ID:
@.***>
Yes we do, let's change it
from coraza-spoa.
Related Issues (20)
- logging issue HOT 3
- Publish artifact on every merge HOT 1
- coraza-spoa / invalid or unsupported Perl syntax / 920120 / crs 3.3.4 HOT 9
- Official container image HOT 8
- TERM signal handling configuration HOT 1
- Looking for a maintainer HOT 3
- Use latest coraza
- E2E test HOT 1
- Move towards magefile
- Replace criteo/haproxy-spoe-go with negasus/haproxy-spoe-go HOT 10
- panic: runtime error: invalid memory address or nil pointer dereference HOT 14
- failed to get transaction from cache HOT 1
- Exception rule fails
- Upgrade to Coraza 3.0
- Add unit tests for internal package
- Adds readme/command to run a example application HOT 1
- Panic on empty Application name
- Adds support for FTW
- Drop logger, logLevel and use coraza's
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from coraza-spoa.