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

For LDAP user sync #1268 #3062

Closed
wants to merge 3 commits into from
Closed

For LDAP user sync #1268 #3062

wants to merge 3 commits into from

Conversation

lafriks
Copy link
Contributor

@lafriks lafriks commented May 5, 2016

Still have to create cron job for regular interval sync

@lafriks
Copy link
Contributor Author

lafriks commented May 5, 2016

Created new cron job with option to update existing user data or just creating new users.
Probably one thing to think about if option not to update users is set probably it would still be needed to deactivate users that are not in LDAP server anymore. Currently it only deactivates users if UpdateUsers option is set to true

This was referenced May 6, 2016
Open
Closed
@unknwon unknwon added the status: waits for review It is waiting to be reviewed by maintainers label May 12, 2016
@ptman
Copy link
Contributor

ptman commented May 20, 2016

Have you thought about updating data on login?

@lafriks
Copy link
Contributor Author

lafriks commented May 20, 2016 •
edited
Loading

Do you mean updating only current users name, surname, email and admin right for user that is logging in or all users?

@ptman
Copy link
Contributor

ptman commented May 20, 2016

Only current

@barracks510
Copy link
Contributor

Can you incorporate an option to disable the removal of old users?

@lafriks
Copy link
Contributor Author

lafriks commented Jul 18, 2016

Old users are not removed, just disabled

@unknwon unknwon added status: review later The pull request is not confident to be review at the moment and removed status: waits for review It is waiting to be reviewed by maintainers labels Aug 17, 2016
@lafriks
Copy link
Contributor Author

lafriks commented Nov 1, 2016

Any progress on merging this?

@lafriks
Copy link
Contributor Author

lafriks commented Mar 18, 2017

?

@unknwon
Copy link
Member

unknwon commented Mar 18, 2017

image

@strk
Copy link
Contributor

strk commented Mar 22, 2017

@ptman would you be up to port this to Gitea ?

@strk strk mentioned this pull request Mar 22, 2017
Closed
7 tasks
@lafriks lafriks mentioned this pull request Apr 10, 2017
Merged
@lafriks
Copy link
Contributor Author

lafriks commented Apr 10, 2017

Pull request have been ported to Gitea, so if you are interested in it get latest changes from there

@lafriks lafriks closed this Sep 25, 2017
@lafriks lafriks deleted the develop branch September 25, 2017 05:49
ethantkoenig pushed a commit to ethantkoenig/gogs that referenced this pull request Dec 11, 2017
@silverwind @lafriks
Fix misalignment issue in repo header (gogs#3062)
8cd616e
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
status: review later The pull request is not confident to be review at the moment
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lafriks @ptman @barracks510 @unknwon @strk