Use WebSocket to achieve granular control of volume transfer

[genkn]

No description provided.

[genkn]

遅くなって申し訳ありません。
テストの fix に苦労しました。

(1) prettier

既存コードもかかっていないところがあります。
比較がより困難になることを防ぐために、こちらではかけていません。
マージ後にそちらで全コード一括、master でお願いできないでしょうか。

(2) ws volume 通信の認可(seriesUid に対する認可)について
- packages/circus-api/src/createApp.ts
- packages/circus-rs/src/server/createServer.ts

現在の定義では authFunctionProvider: req => async seriesUid => true となっているため、
websocket 接続後は、その接続を使用すると認可なく全ての volume にアクセスできてしまいます。

// [createAuthFunctionProvider.ts]
type AuthFunction = (seriesUid: string) => Promise<boolean>;
type AuthFunctionProvider = (req: IncomingMessage /*初回のws接続時の要求*/ ) => AuthFunction;
const createAuthFunctionProvider: FunctionService<AuthFunctionProvider, ...> = ...
export default createAuthFunctionProvider;

のように定義をして、

createWebsocketVolumeConnectionHandlerCreator の依存を調整する方法が良いと思うのですが、
AuthFunctionProvider の認可まわりの実装は、皆目見当がつきません。

こちらの実装はお願いできないでしょうか。

また、認可周りの問題として、
Aさんでログイン⇒ ws接続*1 ⇒ Bさんでログイン
としたときに、*1の接続が残っていた場合、Bさんでログインしているにも関わらず、認可関数はAさんの認証のものが使用されてしまうはずです。
ログアウトに伴うページの離脱が発生しない場合、これは大きな問題になりそうです。
ログアウト時に、全ての接続を切るような対応が必要かもしれません。

[genkn]

先ほどはお打ち合わせありがとうございました。
いくつか、打合せの内容から変更して push しました。

1. websocket の切断について

   • LoginScreen ではなく VolumeLoaderFactoryProvider (packages/circus-web-ui/src/index.tsx) に実装しました。
   • （LoginScreen に変更はありません）

2. setToken をやめてコンストラクタの引数にしました。

よろしくお願いいたします。