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.

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

Get rid of "cache" concept #53

Closed
6 of 7 tasks
tlater-famedly opened this issue Sep 5, 2024 · 1 comment · Fixed by #79
Closed
6 of 7 tasks

Get rid of "cache" concept #53

tlater-famedly opened this issue Sep 5, 2024 · 1 comment · Fixed by #79
Assignees
Labels
enhancement New feature or request

Comments

@tlater-famedly
Copy link
Contributor

tlater-famedly commented Sep 5, 2024

The "cache" is a holdover from ldap-poller, and used to keep track of what changed since the last run of the tool, so that we know when to send update and deletion requests instead of import requests.

While this works for now, and is fine as long as ldap-sync is the only tool performing syncs and the "cache" is persisted, the databases will go out of sync the moment this is not true.

The name "cache" is therefore a big misnomer, since the name implies it can be safely removed at a performance penalty, when in reality doing so breaks the tool. We also currently don't have a good way to get things back into sync when issues (e.g. failed requests) occur because ldap-poller doesn't have an interface to bust the "cache".

We should move away from this.

@tlater-famedly tlater-famedly self-assigned this Sep 5, 2024
@nikzen nikzen added the enhancement New feature or request label Sep 13, 2024
This was referenced Sep 18, 2024
@tlater-famedly
Copy link
Contributor Author

tlater-famedly commented Sep 20, 2024

Waiting for famedly/zitadel-rust-client#15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants