-
Notifications
You must be signed in to change notification settings - Fork 453
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
Possible values of AUTH_INFO_REQUIRED #5674
Comments
@mikhailnov The values of For SMB, the three values you should expect are:
The One other responsibility of a backend that uses
Doing so will make sure that cupsd collects the correct credentials/identity information when a job is submitted so you'll be able to do the right thing in the SMB backend... |
We have been investigating how
smbspool
andsmbspool_krb5_wrapper
fromsamba
must deal withAUTH_INFO_REQUIRED
environmental variable.Discussion was here: https://lists.samba.org/archive/samba-technical/2019-October/134476.html (and other emails in that thread)
I see in
job.c
that AUTH_INFO_REQUIRED may have up to 4 comma-separated values. Its possible values are not docimented and for me were not clear in the source code.I know about following possible values:
Can AUTH_INFO_REQUIRED be null (not set or empty)? If yes, in which cases? What are other possible values and which combinations are possible?
This code tries to understand if kerberos authorization using credentials of the creator of the printing task will be needed or not:
https://git.samba.org/?p=asn/samba.git;a=blob;f=source3/client/smbspool_krb5_wrapper.c;h=85b0d0952a9d190b4e1eb8cac104c21af557c3e3;hb=20dd0308aa072ed656550af487338b50cda9b59f
If it is not needed, then
goto smbspool;
must be done.Currently this code seems not very correct to me because:
none
username,password
ornegotiate
or null otnone
, than that code does not proceed with non-kernerized authenticationSo the main question here is how we must deal with possible values of AUTH_INFO_REQUIRED which are not documented.
Thank you!
The text was updated successfully, but these errors were encountered: