[skywalking-rover] branch main updated: Fixed submitting multiple network profiling tasks with the same uri causing the rover to restart (#70)

[wusheng]

This is an automated email from the ASF dual-hosted git repository.

wusheng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/skywalking-rover.git


The following commit(s) were added to refs/heads/main by this push:
     new 3481481  Fixed submitting multiple network profiling tasks with the 
same uri causing the rover to restart (#70)
3481481 is described below

commit 34814817ec6aaf3f063433d7fb305ff1700a78a7
Author: Aerospace <89571246+hkmdxlf...@users.noreply.github.com>
AuthorDate: Tue Jan 10 18:08:48 2023 +0800

    Fixed submitting multiple network profiling tasks with the same uri causing 
the rover to restart (#70)
---
 CHANGES.md                                                            | 1 +
 pkg/profiling/task/network/analyze/layer7/protocols/http1/sampling.go | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/CHANGES.md b/CHANGES.md
index b6155ec..01abf52 100644
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -8,6 +8,7 @@ Release Notes.
 * Enhance the protocol reader for support long socket data.
 
 #### Bug Fixes
+* Fixed submitting multiple network profiling tasks with the same uri causing 
the rover to restart
 
 #### Documentation
 
diff --git 
a/pkg/profiling/task/network/analyze/layer7/protocols/http1/sampling.go 
b/pkg/profiling/task/network/analyze/layer7/protocols/http1/sampling.go
index 6a98c12..b3964bc 100644
--- a/pkg/profiling/task/network/analyze/layer7/protocols/http1/sampling.go
+++ b/pkg/profiling/task/network/analyze/layer7/protocols/http1/sampling.go
@@ -192,7 +192,7 @@ func (s *SamplingConfig) findMatchesRule(uri string) 
*profiling.NetworkSamplingR
                        continue
                }
                result = rule.Rule
-               s.uriRuleCache.Add(uri, rule)
+               s.uriRuleCache.Add(uri, result)
        }
        return result
 }