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

confusing message when max sessions exceeded #642

Closed
metalefty opened this issue Jan 25, 2017 · 1 comment · Fixed by #2472
Closed

confusing message when max sessions exceeded #642

metalefty opened this issue Jan 25, 2017 · 1 comment · Fixed by #2472
Labels
enhancement sesman

Comments

@metalefty
Copy link
Member

When sessions limit exceeded, logwindow says "login failed for display 0". This is not user friendly. The message should be "sessions limit exceeded".

I looked at sesman, it seems sesman doesn't return the reason of login failure to xrdp. We should think about sesman design. The reason of login failure should be reported to the user not just written to log.

xrdp-mm
@proski
Copy link
Contributor

proski commented Jan 25, 2017

Likewise, failure to execute a backend should be reported to xrdp.

@metalefty metalefty mentioned this issue Oct 13, 2017
Closed
matt335672 added a commit to matt335672/xrdp that referenced this issue Dec 14, 2022
@matt335672
Update PAM auth module for UDS logins
e1b561e 
An extra method auth_uds() is added to the PAM module to
allow a 'struct auth_info' to be created for a UDS login. The PAM stack
is used to check the UDS user can be authorized.

Also, an error code is returned from the auth module rather than a
simple boolean. This allows a more complete status to be communicated
to the user. See neutrinolabs#1921
and also neutrinolabs#909 and neutrinolabs#642
matt335672 added a commit to matt335672/xrdp that referenced this issue Dec 14, 2022
@matt335672
Update PAM auth module for UDS logins
71a13b5 
An extra method auth_uds() is added to the PAM module to
allow a 'struct auth_info' to be created for a UDS login. The PAM stack
is used to check the UDS user can be authorized.

Also, an error code is returned from the auth module rather than a
simple boolean. This allows a more complete status to be communicated
to the user. See neutrinolabs#1921
and also neutrinolabs#909 and neutrinolabs#642
@matt335672 matt335672 mentioned this issue Dec 14, 2022
Merged
matt335672 added a commit to matt335672/xrdp that referenced this issue Dec 22, 2022
@matt335672
Update PAM auth module for UDS logins
2a3cec4 
An extra method auth_uds() is added to the PAM module to
allow a 'struct auth_info' to be created for a UDS login. The PAM stack
is used to check the UDS user can be authorized.

Also, an error code is returned from the auth module rather than a
simple boolean. This allows a more complete status to be communicated
to the user. See neutrinolabs#1921
and also neutrinolabs#909 and neutrinolabs#642
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement sesman
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement uds authentication matt335672/xrdp
2 participants
@metalefty @proski