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

onLoginAsRoot scripts not always executed #69

Open
kiarn opened this issue Jan 16, 2023 · 4 comments
Open

onLoginAsRoot scripts not always executed #69

kiarn opened this issue Jan 16, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@kiarn
Copy link
Member

kiarn commented Jan 16, 2023

Hello @dorianim

I've just see that the onLoginAsRoot scripts are not always executed, I ran today in this error:

Jan 16 17:15:52 r10106 linuxmuster-linuxclient7: [INFO] ====== onLoginLogoutAsRoot started with PAM_TYPE=open_session PAM_RHOST=None PAM_RUSER=None PAM_SERVICE=lightdm PAM_TTY=:0 PAM_USER=test1 ======
Jan 16 17:15:52 r10106 linuxmuster-linuxclient7: [DEBUG] Calculating mountpoint of //server.linuxmuster.lan/sysvol
Jan 16 17:15:52 r10106 linuxmuster-linuxclient7: [DEBUG] Trying to mount //server.linuxmuster.lan/sysvol to /srv/samba/R10106$/sysvol
Jan 16 17:15:52 r10106 linuxmuster-linuxclient7: [DEBUG] * Creating directory...
Jan 16 17:15:52 r10106 linuxmuster-linuxclient7: [WARNING] * The target directory already exists, proceeding anyway!
Jan 16 17:15:52 r10106 linuxmuster-linuxclient7: [ERROR] === An exception occurred ===
Jan 16 17:15:52 r10106 linuxmuster-linuxclient7: [ERROR] sequence item 0: expected str instance, NoneType found
Jan 16 17:15:52 r10106 linuxmuster-linuxclient7: [ERROR] === end exception ===
...
Jan 16 17:15:53 r10106 linuxmuster-linuxclient7: [INFO] ====== onLogin started ======

Unfortunately I don't have enough time to test more today, I will do it later if you don't see directly a reason for this error. I think this problem occurs each time the mount of sysvol fails.

Thanks

Arnaud

@dorianim
Copy link
Member

dorianim commented Jan 16, 2023

I haven't seen that before. Thanks for reporting :)
I'll look into it when I have time, but let me know when you find the cause.

@dorianim dorianim added the bug Something isn't working label Jan 16, 2023
@dorianim
Copy link
Member

@kiarn you can uncomment this line for easier debugging:

@kiarn
Copy link
Member Author

kiarn commented Jan 20, 2023

Hello @dorianim ,

I'm sorry, I have currently not enough time to debug it, but I will not forget it.
Is there for me a quick fix/patch to apply in order to let the onLoginAsRoot scripts be executed, even if sysvol is not mounted ? It would at least solve some problems on our network.

Cheers

Arnaud

@dorianim
Copy link
Member

Please uncomment the line from logging.py that I mentioned and post the traceback. Otherwise, It's really hard to debug this, since I can't reproduce it.

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

No branches or pull requests

2 participants