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

missing handler for opn_user #89

Closed
zerwes opened this issue Aug 6, 2024 · 4 comments · Fixed by #94
Closed

missing handler for opn_user #89

zerwes opened this issue Aug 6, 2024 · 4 comments · Fixed by #94
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@zerwes
Copy link
Collaborator

zerwes commented Aug 6, 2024

at least the authorizedkeyskey need some trigger to put the ssh keys into .ssh/authorized_keys

wip for #12

@zerwes zerwes added bug Something isn't working enhancement New feature or request labels Aug 6, 2024
@zerwes zerwes self-assigned this Aug 6, 2024
@crpb
Copy link

crpb commented Aug 20, 2024

@zerwes
Copy link
Collaborator Author

zerwes commented Aug 21, 2024

Hello @crpb
maybe local_sync_accounts() (https://github.com/opnsense/core/blob/24.7.1/src/etc/inc/auth.inc#L243) is more what we are looking for?
Not sure when I will find the time for this, so if you finde some new hints, you eare welcome to share them, all contributions are wellcome

@zerwes
Copy link
Collaborator Author

zerwes commented Aug 21, 2024

ok; just did a q&dirty test with apparently a good result (at least regarding the ssh keys)

@crpb if you find the time and like to do some testing, could you please create a script
synclocalaccounts.php (on the opnsense device as root and chmod it to 700)

#!/usr/local/bin/php
<?php

require_once 'config.inc';
require_once 'auth.inc';

local_sync_accounts();

?>

deploy some user related changes via ansible to the /conf/config.xml and run the script manually afterwards and test if the user settings changed in the xml are pushed through?

Would be somehow nice if 3 lines of code will do the trick 😃 ...

@zerwes
Copy link
Collaborator Author

zerwes commented Aug 21, 2024

@crpb testing on the branch localaccounthandler ( #94 ) should be even easier, as the 3line-handler is already implemented there ...

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

Successfully merging a pull request may close this issue.

5 participants