Bug: flags field is missing on RLS-Realtime

[Vinzent03]

See #74 (comment)

[Vinzent03]

RLS-Realtime doesn't ship the flags field anymore. supabase/realtime#210 (comment)

So it isn't possible to generate the primaryKeys list. See here. I think we should mark it to be only compatible with old supabase projects, because all new supabase project use RLS-realtime if I understand @w3b6x9 correctly.

The problem is now that stream depends on it to replace existing data. See here. payload.primaryKeys is empty, which causes always the first value in _streamData to be replaced. I think the only way to fix this is that the user has to enter the primary key manually.

[JPGarCar]

Hi @Vinzent03 wondering if this client is using Realtime RLS or the old Realtime implementation, as it seems like Realtime RLS is going to be the default for all supabase projects.

[Vinzent03]

@JPGarCar I don't really know what you mean, but this client should work for both systems for backwards compatibility. I don't think there are many changes needed to be rls-realtime compatible. The only issue I experienced is this one.

[Vinzent03]

Any updates? I think the stream method is crucial for many apps. I can submit a pr, but I want to make sure my idea of solving this (the user has to enter the primary key manually) is accepted by you.

[nitheshb]

@Vinzent03 the issue of appending changed data to the top of the output still exists even after modifying my table structure to enter primary key column values manually without using any of the default auto gen functions. More over if any of the row deletions is also not being recognized. Please suggest.

[Vinzent03]

@nitheshb Sorry but I don't understand what you want to tell me at all. With "the user has to enter the primary key manually" I meant the primary key name has to be given to the stream method as parameter, because that information is not coming from the backend anymore. I should have said dev instead of user, maybe.

[nitheshb]

@Vinzent03 can u please share any syntax on how to pass the key as a parameter to stream. As iam using this over steamBuilder Widget in flutter.

[Vinzent03]

@nitheshb

supabase.from('my_table').stream("my_key").execute();

[nitheshb]

@Vinzent03 The above syntax is not valid. As stream method doesn't take any parameters as per version : supabase: ^0.2.10.

Syntax I tried: Supabase.instance.client.from('test').stream("my_key").execute();
Error Msg: Too many position arguments: 0 expected but 1 found .

[Vinzent03]

I said

I can submit a pr, but I want to make sure my idea of solving this (the user has to enter the primary key manually) is accepted by you.

and this issue is still open, so it's not fixed. You asked for a syntax example. It's just a proposal to fix this issue. That parameter doesn't exist, because it's the way I would solve this issue, but I don't want to submit a pr if the supabase-comunity devs don't like that way of solving the missing information about primary keys.

[nitheshb]

Thank you I got it.it make sense. Sorry for my ignorance.

[dshukertjr]

So sorry for coming in late. Had been dealing with some personal issues lately.

@Vinzent03

Any updates? I think the stream method is crucial for many apps. I can submit a pr, but I want to make sure my idea of solving this (the user has to enter the primary key manually) is accepted by you.

I was thinking how we can solve this, and I love this route.