share to mixed group not arriving

SURFnet / rd-sram-integration

Research Drive / SURF Research Access Management Integration

2 stars 3 forks source link

share to mixed group not arriving #213

Closed michielbdejong closed 1 year ago

michielbdejong commented 1 year ago

follow https://github.com/suRFnet/rd-sram-integration#testing-environment
as einstein on oc1.docker, create a folder asdf and share it with 'test group ... '
as marie on oc2.docker you'll see that no share arrived

shokri-navid commented 1 year ago

🤔 So why marie did not receive any share? I tested this happy scenario dozen times and it was working. Maybe the marie is not a member of the remote group at all.

michielbdejong commented 1 year ago

if you were unable to reproduce this then let's do a screenshare session where i talk you through it!

michielbdejong commented 1 year ago

I tested this happy scenario dozen times

just to clarify, I ran into this bug with this version of the code which was published just 25 minutes before your comment. Maybe you are talking about a previous version of the code?

michielbdejong commented 1 year ago

where "previous" may also be a different branch of dev-stock, a different branch of the opencloudmesh app, a different branch of the federatedgroups app, or a different branch of owncloud/core. All these things influence it, so as we found this morning, we need to make sure we all use the same revision of the same fork! :)

navid-shokri commented 1 year ago

:information_source:

So, I guess this image can prof my forecast but I will try to show more symptoms:

As it is obvious in the above image marei is just a member of the admin group. So I will share the init.sh file with the test group:

and yes Marie is not notified because she is not enlisted in the test group membership. but, if you look into the maria2.docker

the new record is added to the corresponding table.

navid-shokri commented 1 year ago

:information_source: So, I will add Marie to the Test group: :tada: So as expected Marie will be notified about the new incoming share:

navid-shokri commented 1 year ago

:information_source: and I will share the shell file with Marie when she is a member of the test group

and she is notified directly without any problem:

michielbdejong commented 1 year ago

Ah right! I had forgotten about https://github.com/pondersource/dev-stock/blob/main/scripts/testing-rd-sram.sh#L98-L99 - thanks!