Fix file matching in Aliyun OSS GetContent (#4692)

Was comparing path.Base(f) against full fileName, so they'd never
match. Fixed by comparing basenames on both sides and added found
flag with break to exit early.

Author: utafrali

historyserver/pkg/storage/aliyunoss/ray/ray.go

@@ -206,7 +206,7 @@ func (r *RayLogsHandler) GetContent(clusterId string, fileName string) io.Reader
 		allFiles := r._listFiles(clusterId+"/"+path.Dir(fileName), "", false)
 		found := false
 		for _, f := range allFiles {
-			if path.Base(f) == fileName {
+			if path.Base(f) == path.Base(fileName) {
 				logrus.Infof("Get object %s info success", f)
 				result, err = r.OssClient.GetObject(ctx, &oss.GetObjectRequest{
 					Bucket: oss.Ptr(r.OssBucket),
@@ -216,6 +216,8 @@ func (r *RayLogsHandler) GetContent(clusterId string, fileName string) io.Reader
 					logrus.Errorf("Failed to get object %s: %v", f, err)
 					return nil
 				}
+				found = true
+				break
 			}
 		}
 		if !found {

historyserver/pkg/storage/aliyunoss/ray/ray_test.go

@@ -30,6 +30,21 @@ func TestTrim(t *testing.T) {
 	t.Logf("test_path_join [%s]", test_path_join)
 }
 
+func TestGetContentPathComparison(t *testing.T) {
+	// fileName is a full path like the caller passes
+	fileName := "session_2026-04-09_06-41-42_754637_1/logs/abc123/events/event_RAYLET.log"
+	// f is a full key returned by _listFiles (onlyBase=false)
+	f := "session_2026-04-09_06-41-42_754637_1/logs/abc123/events/event_RAYLET.log"
+
+	if path.Base(f) != path.Base(fileName) {
+		t.Errorf("expected path.Base(%q) == path.Base(%q)", f, fileName)
+	}
+	// Verify old buggy comparison would fail
+	if path.Base(f) == fileName {
+		t.Errorf("old comparison should not match: path.Base(%q) == %q", f, fileName)
+	}
+}
+
 func TestWalk(t *testing.T) {
 	watchPath := "/tmp/ray/test/LLogs/"
 	filepath.Walk(watchPath, func(path string, info os.FileInfo, err error) error {