SURFnet / rd-sram-integration

Research Drive / SURF Research Access Management Integration
2 stars 3 forks source link

Group is created but users are not #190

Closed michielbdejong closed 1 year ago

michielbdejong commented 1 year ago

See https://github.com/SURFnet/rd-sram-integration/issues/184#issuecomment-1584221776

michielbdejong commented 1 year ago

This was due to database replication. If Tom removes the db proxy then this problem goes away. To fix it in general (so that it also works with a db proxy) we should use the cache in the group backend. We fill it here: https://github.com/SURFnet/rd-sram-integration/blob/2207762/federatedgroups/lib/GroupBackend.php#L97 But we don't read it here: https://github.com/SURFnet/rd-sram-integration/blob/2207762/federatedgroups/lib/GroupBackend.php#L148

navid-shokri commented 1 year ago

can we close this Issue? @michielbdejong

thepeak99 commented 1 year ago

The customer wants to use it with a DB cluster. We have to check that it works.

thepeak99 commented 1 year ago

We should not assume that our inserts will pop up in subsequent selects.

Also we need to create a test for this.

thepeak99 commented 1 year ago

duplicate of #197