Add changes

Author: piyushroshan

Diff for: experimental/types/value_metadata.go

@@ -96,7 +96,7 @@ func NewDataMetadataList() DataMetadataList {
 
 func (v *DataMetadataList) EvaluateMetadata(data string) {
 	// we do the analysis only once
-	if !v.evaluated {
+	if v != nil && !v.evaluated {
 		v.metadata = make(map[DataMetadata]bool)
 		v.evaluateBoolean(data)
 		v.evaluateNumeric(data)

Diff for: internal/corazarules/rule_match.go

@@ -66,6 +66,9 @@ func (m MatchData) ChainLevel() int {
 
 func (m *MatchData) DataMetadata() experimentalTypes.DataMetadataList {
 	// Evaluate the metadata if it's not set
+	if m.Metadata_ == nil {
+		m.Metadata_ = &experimentalTypes.DataMetadataList{}
+	}
 	m.Metadata_.EvaluateMetadata(m.Value_)
 	return *m.Metadata_
 }

Diff for: internal/seclang/parser_test.go

@@ -154,6 +154,7 @@ func TestAllowedMetadataTagsInspectionEnabled(t *testing.T) {
 		t.Error(err)
 	}
 	tx := waf.NewTransaction()
+	tx.SetMetadataInspection(true)
 	tx.AddRequestHeader("Content-Type", "application/json")
 	tx.ProcessURI("http://localhost/test.php", "GET", "1.1")
 	tx.ProcessRequestHeaders()
@@ -210,6 +211,7 @@ func BenchmarkAllowedMetadataTagsInspectionEnabled(b *testing.B) {
 	b.ResetTimer()
 	for i := 0; i < b.N; i++ {
 		tx := waf.NewTransaction()
+		tx.SetMetadataInspection(true)
 		tx.AddRequestHeader("Content-Type", "application/json")
 		tx.ProcessURI("http://localhost/test.php", "GET", "1.1")
 		tx.ProcessRequestHeaders()
@@ -267,6 +269,7 @@ func BenchmarkAllowedMetadataTagsInspectionDisabled(b *testing.B) {
 	b.ResetTimer()
 	for i := 0; i < b.N; i++ {
 		tx := waf.NewTransaction()
+		tx.SetMetadataInspection(false)
 		tx.AddRequestHeader("Content-Type", "application/json")
 		tx.SetMetadataInspection(false)
 		tx.ProcessURI("http://localhost/test.php", "GET", "1.1")