fix(labels): Prevent duplicate labels POST call (#424) (#425)

* fix(labels): Prevent duplicate labels POST call

* Refresh labels only for same target

(cherry picked from commit 5a9b7be)

Co-authored-by: Janelle Law <[email protected]>

Author: mergify[bot]

src/app/Recordings/ActiveRecordingsTable.tsx

@@ -193,9 +193,21 @@ export const ActiveRecordingsTable: React.FunctionComponent<ActiveRecordingsTabl
 
   React.useEffect(() => {
     addSubscription(
-      context.notificationChannel.messages(NotificationCategory.RecordingMetadataUpdated)
-        .subscribe(v => setRecordings(old => old.map(
-          o => o.name == v.message.recordingName ? { ...o, metadata: { labels: v.message.metadata.labels } } : o)))
+      combineLatest([
+      context.target.target(),
+      context.notificationChannel.messages(NotificationCategory.RecordingMetadataUpdated),
+    ])
+      .subscribe(parts => {
+        const currentTarget = parts[0];
+        const event = parts[1];
+        if (currentTarget.connectUrl != event.message.target) {
+          return;
+        }
+        setRecordings(old => old.map(
+          o => o.name == event.message.recordingName 
+            ? { ...o, metadata: { labels: event.message.metadata.labels } } 
+            : o));
+      })
     );
   }, [addSubscription, context, context.notificationChannel, setRecordings]);
 

src/app/Shared/Services/Api.service.tsx

@@ -536,7 +536,9 @@ export class ApiService {
   }
 
   postTargetRecordingMetadata(recordingName: string, labels: RecordingLabel[]): Observable<string> {
-    return this.target.target().pipe(concatMap(target =>
+    return this.target.target().pipe(
+      first(),
+      concatMap(target =>
         this.sendRequest(
           'beta', `targets/${encodeURIComponent(target.connectUrl)}/recordings/${encodeURIComponent(recordingName)}/metadata/labels`,
           {