Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Confirm sync operates as expected #208

Open
felixoi opened this issue Aug 5, 2018 · 3 comments
Open

Confirm sync operates as expected #208

felixoi opened this issue Aug 5, 2018 · 3 comments

Comments

@felixoi
Copy link
Member

felixoi commented Aug 5, 2018 •
edited
Loading

Moved for @mbaxter from SpongePowered/Ore#604 (so note that I don't have the same views as him)

When Ore and Auth are both online:

  • Staff role addition is synchronized within a minute.
  • Staff role removal is synchronized within a minute.
  • Ore handles any response Auth sends at it properly, without exception thrown.

When Ore is offline and Auth is online:

  • Auth holds on to role changes and synchronizes within a minute of Ore coming back.

When Ore throws an error at Auth about role synchronization despite our best attempts above:

  • Auth or Ore (or both) send angry, urgent messages to people who can promptly resolve this.
lukegb added a commit that referenced this issue Aug 12, 2018
@lukegb
sso: send_update_ping if groups list changes for a user (#208)
8bcd444 
This happens if, for instance, the groups are modified from the
Group admin page, and not from the User admin page.
@felixoi
Copy link
Member Author

felixoi commented Mar 11, 2019

I'd guess this can be closed since sync SSO is working as expected.

@lukegb
Copy link
Collaborator

lukegb commented Mar 12, 2019

Disagree: I haven't done this yet. I need to put sync into a background queue, but that's not yet done.

@lukegb lukegb reopened this Mar 12, 2019
@felixoi
Copy link
Member Author

felixoi commented Apr 21, 2020

This is done now or?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lukegb @progwml6 @felixoi