Отписаться от Observables внутри сервисов?

Глядя на примеры кода, я часто вижу случай, когда наблюдаемые внутри сервисов не отписываются.

Вы смотрите на ошибку.

Есть ли причина, по которой вы не отпишетесь от наблюдаемого this.isLoggedIn$ в этом случае?

Если вы хотите утечку памяти.

Или приведенный выше пример просто плохой код, приводящий к утечкам памяти?

Да, это утечка памяти. Функция подписки будет продолжать выполняться после того, как объект, на который ссылается this, будет уничтожен. Это может никогда не произойти, если этот объект никогда не будет уничтожен, но это пример кода, который не пройдет модульный тест, в котором объекты создаются временно.

Объекты, помеченные @Injectable(), часто ведут себя как синглтоны или имеют слабые ссылки. Это может работать какое-то время, но как только вы используете его во временном случае, произойдет утечка памяти.

Для базовых служб отмена подписки на самом деле не требуется, потому что служба существует до тех пор, пока живет приложение, а служба является одноэлементной. Проблема в том, что вы должны убедиться, что вы всегда используете его как синглтон (для лучших решений читайте ниже)

Для сервисов-компонентов отписку следует размещать внутри ngOnDestroy сервиса, потому что сервисы на самом деле могут реализовывать этот хук NgOnDestroy. Лучшим способом было бы использовать канал takeUntil(this.destroy) и испускать его при уничтожении.

Еще лучше было бы использовать канал async внутри шаблона и никогда напрямую не подписываться на подобные вещи.

С другой стороны, в вашей защите вы можете использовать трубу take(1), это приведет к первому выбросу и немедленной отмене подписки без необходимости для вас, это изменит ваш пример на этот. Как видите, подписки внутри кода нет:

export class AuthGuard implements CanActivate {

  constructor(private authService: AuthService, private router: Router) {}

  canActivate(route: ActivatedRouteSnapshot, state: RouterStateSnapshot) {
    return this.authService.isLoggedIn().pipe(
      take(1),
      tap((loggedIn) => {
        if (!loggedIn) {
          this.router.navigate(['login'])
        }
      })
    );    
  }
}

В итоге постарайтесь, чтобы ваше приложение было потоком Observables, используя асинхронные внутренние шаблоны и функцию канала rxjs вместо подписки и хранения дополнительных результатов. Чтобы isLoggedIn() выдавал последний результат при подписке, вы можете использовать канал shareReplay() или сделать его BehaviourSubject для начала.