Cleanup

We must not expose private actor objects to clients
Just make 2 GET requests on channel/accounts instead

2 files changed, 47 insertions, 0 deletions

diff --git a/client/src/app/shared/shared-main/router/actor-redirect-guard.service.ts b/client/src/app/shared/shared-main/router/actor-redirect-guard.service.ts
new file mode 100644
index 000000000..49d61f945
--- /dev/null
+++ b/client/src/app/shared/shared-main/router/actor-redirect-guard.service.ts
@@ -0,0 +1,46 @@
+import { forkJoin, of } from 'rxjs'
+import { catchError, map } from 'rxjs/operators'
+import { Injectable } from '@angular/core'
+import { ActivatedRouteSnapshot, CanActivate, Router } from '@angular/router'
+import { AccountService } from '../account'
+import { VideoChannelService } from '../video-channel'
+
+@Injectable()
+export class ActorRedirectGuard implements CanActivate {
+
+  constructor (
+    private router: Router,
+    private accountService: AccountService,
+    private channelService: VideoChannelService
+  ) {}
+
+  canActivate (route: ActivatedRouteSnapshot) {
+    const actorName = route.params.actorName
+
+    return forkJoin([
+      this.accountService.getAccount(actorName).pipe(this.orUndefined()),
+      this.channelService.getVideoChannel(actorName).pipe(this.orUndefined())
+    ]).pipe(
+      map(([ account, channel ]) => {
+        if (!account && !channel) {
+          this.router.navigate([ '/404' ])
+          return false
+        }
+
+        if (account) {
+          this.router.navigate([ `/a/${actorName}` ], { skipLocationChange: true })
+        }
+
+        if (channel) {
+          this.router.navigate([ `/c/${actorName}` ], { skipLocationChange: true })
+        }
+
+        return true
+      })
+    )
+  }
+
+  private orUndefined () {
+    return catchError(() => of(undefined))
+  }
+}
diff --git a/client/src/app/shared/shared-main/router/index.ts b/client/src/app/shared/shared-main/router/index.ts
new file mode 100644
index 000000000..f4000b674
--- /dev/null
+++ b/client/src/app/shared/shared-main/router/index.ts
@@ -0,0 +1 @@
+export * from './actor-redirect-guard.service'