Bug 28200 - Passwords Module honorieren autosearch=0 nicht
Passwords Module honorieren autosearch=0 nicht
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC
UCS@school 3.0
Other Linux
: P5 normal (vote)
: UCS@school 3.1
Assigned To: Jascha Geerds
Dirk Wiesenthal
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-08-13 17:05 CEST by Janis Meybohm
Modified: 2013-02-15 17:50 CET (History)
4 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): UCS Performance
Max CVSS v3 score:


Attachments
Quick & Dirty (420 bytes, patch)
2012-08-15 15:15 CEST, Dirk Wiesenthal
Details | Diff
Configurable by UCR (1.66 KB, patch)
2012-08-15 16:35 CEST, Dirk Wiesenthal
Details | Diff
Configurable by UCR (patch fixed (1.66 KB, patch)
2012-08-16 09:11 CEST, Janis Meybohm
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Janis Meybohm univentionstaff 2012-08-13 17:05:21 CEST
Auch mit "directory/manager/web/modules/autosearch=0" suchen die Module standardmäßig erstmal alle Benutzer (im Suchfeld "fehlt" hier auch das Asterisk aus den anderen UMC Modulen).

Die UCR Variable sollte auch hier honoriert und der autosearch nicht gestartet werden. Ich denke in den meisten Fällen ist die mind. die Klasse oder auch ein Suchfilter bekannt den der Anwender eh eingeben würde.
Comment 1 Stefan Gohmann univentionstaff 2012-08-14 06:18:58 CEST
Zunächst auf 3.1. Wenn das kritisch ist, dann bitte melden, dann könnten wir einen Patch an den Bug hängen.
Comment 2 Janis Meybohm univentionstaff 2012-08-14 16:29:19 CEST
(In reply to comment #1)
> Zunächst auf 3.1. Wenn das kritisch ist, dann bitte melden, dann könnten wir
> einen Patch an den Bug hängen.

Ich denke ein Patch wäre schön, dann kann das bei akuten Problemen in Kundenumgebungen direkt angepasst werden.
Comment 3 Stefan Gohmann univentionstaff 2012-08-14 16:40:35 CEST
OK, nach der Patch Erstellung kann der Bug zunächst wieder auf UCS@school 3.1 gesetzt werden.

Wenn das Paket eh in einem Errata aktualisiert wird, dann sollte der Patch einfließen.
Comment 4 Dirk Wiesenthal univentionstaff 2012-08-15 15:15:51 CEST
Created attachment 4594 [details]
Quick & Dirty

Je nachdem, wie dringend die Sache ist, kann man es mit der Brechstange lösen (Patch anbei).

Das ist natürlich nicht der Weisheit letzter Schluss, aber das über UCR konfigurierbar zu machen, bringt irgendwelche hinterhältigen Nebenläufigkeitsprobleme mit sich.

Deshalb gibt es bis jetzt auch noch keinen "echten" Patch. Auf die Schnelle sollte Patch oben aber funktionieren.
Comment 5 Dirk Wiesenthal univentionstaff 2012-08-15 16:35:48 CEST
Created attachment 4596 [details]
Configurable by UCR

So sollte es korrekt sein. Man kann nun dem Modul das Autosearch verbieten über die Variablen

directory/manager/web/modules/users/user/search/autosearch
bzw.
directory/manager/web/modules/autosearch

(ersteres sticht zweiteres).
Comment 6 rolandb 2012-08-15 17:35:06 CEST
Hallo Leute

Kann mir jemand erklären wie ich diesen Patch auf dem School-DC installiere? 

find /usr/ -iname "*schoolusers.js*"
/usr/share/univention-management-console-frontend/js/umc/modules/schoolusers.js
/usr/share/univention-management-console-frontend/js/umc/modules/i18n/de/schoolusers.json


Ich nehme an es sollte die Datei /usr/share/univention-management-console-frontend/js/umc/modules/schoolusers.js damit gepatch werden? Leider stimmt dieser Pfad nicht mit dem Patch überein.

Vielen Dank für die Hilfe.

mfg, Roland
Comment 7 Alexander Kläser univentionstaff 2012-08-15 17:41:31 CEST
(In reply to comment #6)
> Kann mir jemand erklären wie ich diesen Patch auf dem School-DC installiere? 
> 
> find /usr/ -iname "*schoolusers.js*"
> /usr/share/univention-management-console-frontend/js/umc/modules/schoolusers.js
> /usr/share/univention-management-console-frontend/js/umc/modules/i18n/de/schoolusers.json
> 
> Ich nehme an es sollte die Datei
> /usr/share/univention-management-console-frontend/js/umc/modules/schoolusers.js
> damit gepatch werden? Leider stimmt dieser Pfad nicht mit dem Patch überein.

Ja, genau. Der folgende Befehl sollte den Patch korrekt einspielen:

> patch /usr/share/univention-management-console-frontend/js/umc/modules/schoolusers.js bug28200.patch
Comment 8 Dirk Wiesenthal univentionstaff 2012-08-15 18:05:00 CEST
Wenn dieser Patch nicht über die Paketverwaltung eingespielt wird, fehlt ein wichtiger Schritt, der ansonsten automatisch ausgeführt worden wäre:

Der timestamp in der index.html wird nicht aktualisiert. Er wird benutzt um zu überprüfen, ob der gesamte JS-Code vom Browser heruntergeladen werden muss (das dauert eine ganze Weile und sollte wenn möglich vermieden werden).

Jedenfalls ist der timestamp nach dem händischen Patch immer noch der gleiche, obwohl UMC aktualisiert wurde.

Also noch zwei Befehle absetzen, damit das Caching-Problem nicht auftaucht.

source /usr/share/univention-lib/umc.sh
umc_frontend_new_hash
Comment 9 Janis Meybohm univentionstaff 2012-08-16 09:11:04 CEST
Created attachment 4599 [details]
Configurable by UCR (patch fixed

Ursprünglicher Patch war defekt:
patch /usr/share/univention-management-console-frontend/js/umc/modules/schoolusers.js ucsschool-umc-modul-passwort-2012.08.15-Buzilla28200.patch
patching file /usr/share/univention-management-console-frontend/js/umc/modules/schoolusers.js
patch: **** malformed patch at line 62:
Comment 10 Jascha Geerds univentionstaff 2013-01-16 17:18:59 CET
Der Patch brauchte nicht übernommen werden, da dieser bereits im 3.0er Branch eingepflegt wurde. Allerdings geschah dies erst nachdem UCS@school 3.0 released wurde, insofern ist der Fix noch nicht veröffentlicht (auch nicht via errata).

Da ist in der Vergangenheit wohl etwas schief gelaufen. Ich habe lediglich die Änderungen überprüft und einen Changelog-Eintrag dafür geschrieben.

Fixed in ucs-school-umc-users (8.0.6-1 bzw. 8.0.7-1)
Comment 11 Dirk Wiesenthal univentionstaff 2013-01-17 17:50:56 CET
(In reply to comment #10)
> Da ist in der Vergangenheit wohl etwas schief gelaufen. Ich habe lediglich die
> Änderungen überprüft und einen Changelog-Eintrag dafür geschrieben.

Das sollte wohl so sein. Siehe Comment 3

Funktioniert jedenfalls einwandfrei
Changelog Ok
Comment 12 Sönke Schwardt-Krummrich univentionstaff 2013-02-15 17:50:55 CET
UCS@school 3.1 has been released: 
 http://forum.univention.de/viewtopic.php?f=26&t=2364

If this error occurs again, please use "Clone This Bug".