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

implement local account handler #94

Merged
merged 5 commits into from
Aug 22, 2024
Merged

implement local account handler #94

merged 5 commits into from
Aug 22, 2024

Conversation

zerwes
Copy link
Collaborator

@zerwes zerwes commented Aug 21, 2024

closes #89

@zerwes zerwes mentioned this pull request Aug 21, 2024
Closed
@zerwes zerwes self-assigned this Aug 21, 2024
@zerwes zerwes added the enhancement New feature or request label Aug 21, 2024
@zerwes
Copy link
Collaborator Author

zerwes commented Aug 22, 2024

lgtm
with changes similar to those documetet in e93569c I could add a new user and was able to login via webUI and ssh.

only restriction I found: seems like changing the shell after the creation of the user is not working (put that is another story / issue, to be told and fixed in another PR ...)

@zerwes zerwes merged commit f1492b8 into main Aug 22, 2024
4 checks passed
@zerwes zerwes deleted the localaccounthandler branch August 22, 2024 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mnaier mnaier Awaiting requested review from mnaier

@fnateghi fnateghi Awaiting requested review from fnateghi

@potzblitzzz potzblitzzz Awaiting requested review from potzblitzzz

Assignees

@zerwes zerwes

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

missing handler for opn_user
1 participant
@zerwes