Drop Transaction.ProcessRequest

[jcchavezs]

Summary

I'd like to propose to drop Transaction.ProcessRequest from the coraza package (see

coraza/transaction.go

Line 452 in ddbce89

func (tx *Transaction) ProcessRequest(req *http.Request) (*types.Interruption, error) {

). The reason is that it is too specific and makes assumptions about the usage of the library in the core package. Ideally this kind of specific helpers should go to another package or be an example.

On a side not, net/http package isn't supported by tinygo making this inconvenient, that was my original motivation for this request but if I could ignore this function in tiny_go, I believe this is a too specific function.

cc @jptosso @fzipi

[jptosso]

I agree with moving this function and all transaction helpers to another package, like coraza/v3/utils/http

[github-actions]

This issue is stale because it has been open for 30 days with no activity.

[jptosso]

Implemented in v3 #296