Univention Bugzilla – Bug 38423
printjobs not displayed for students with upper case letters in UID (e.g. if printed via lpr)
Last modified: 2015-11-11 14:22:22 CET
This bug has not been completely fixed with UCS@school 4.0 R2, but the user's experience is much better now (since most users are using a Windows client for printing). Given the case that there is a user with some capital letters in its uid, e.g. "JohnDoe", then it depends on the printing tools, if capital letters are used for the cups-pdf spooling directory (/var/spool/cups-pdf/$UID). These are the resulting directory names depending on the print tool: smbclient: /var/spool/cups-pdf/johndoe/ lpr: /var/spool/cups-pdf/JohnDoe/ So the printer moderation should check any directory that matches case-insensitive to the user's UID. +++ This bug was initially created as a clone of Bug #34934 +++ printjobs of students which have differences in their lower/uppercase username aren't displayed in the printermoderation module.
Created attachment 6869 [details] patch patch (untested, i hope copy.deepcopy() works).
Fixed with a nicer patch, (also in 3.2 R2 via Bug #38230). Fix: svn r60413 ucs-school-umc-printermoderation 11.0.2-1
> path_username = dict((self._get_path(username, ''), username) for username in \ (username, username.lower(), username.upper())) I don't think that this change covers all variants. The code will currectly check 3 directories: - $username as stored in LDAP - $username in lower case - $username in upper case If "JohnDoe" is stored in LDAP but a tool (Windows-Client/smbprint/lpr/…) created the cups-pdf directory "JoHndOE", the printjobs within the directory are ignored. Despite that, the current version seems to be better that the current state. A functional test is currently missing.
(In reply to Sönke Schwardt-Krummrich from comment #3) > I don't think that this change covers all variants. The code will currectly > check 3 directories: > - $username as stored in LDAP > - $username in lower case > - $username in upper case This was the case. The code has been altered so that the module now actively checks for all existing variants of the usernames. > A functional test is currently missing. Functional test was successful. Code change has been reviewed by fbest. Changelog entry been added to README_UPDATE_*
UCS@school 4.0 R2 v3 has been released. If this error occurs again, please use "Clone This Bug".