Bug 28182 – fehlerhafte Überprüfung auf request.options typ in umc-server response

Bug 28182 - fehlerhafte Überprüfung auf request.options typ in umc-server response


Summary:	fehlerhafte Überprüfung auf request.options typ in umc-server response

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	UMC (Generic)
Version:	UCS 3.0
Hardware:	Other Linux

Importance:	P3 normal (vote)
Target Milestone:	UCS 3.1
Assigned To:	Florian Best
QA Contact:	Dirk Wiesenthal

URL:
Keywords:	interim-1

Depends on:	28102
Blocks:
	Show dependency tree / graph

Reported:	2012-08-10 12:54 CEST by Florian Best
Modified:	2012-12-12 21:08 CET (History)
CC List:	2 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):
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Florian Best

2012-08-10 12:54:57 CEST

Durch den fix können folgefehler auftreten:

if 'options' in body and isinstance(body, dict):

würde fehlschlagen, wenn body ein Datentyp ist, der kein "in" unterstützt.
Momentan gibt es (soweit ich weiß) kein Modul, in dem das vorkommt.

+++ This bug was initially created as a clone of Bug #28102 +++
………
line 292, in response data = str( msg ) File
'/usr/lib/pymodules/python2.6/univention/management/console/protocol/message.py',
line 311, in __str__ del body['options'] TypeError: 'str' object does not
support item deletion " }

Comment 1 Florian Best

2012-08-10 12:57:12 CEST

fixed in univention-management-console (5.0.12-1)

Comment 2 Dirk Wiesenthal

2012-09-13 16:34:24 CEST

Nur manuell nachzustellen, aber der Fehler war tatsächlich bloß ein kleiner Dreher. Verified durch Codereview, da aufgefallen auch nur durch Codereview und ohne konkreten Anlass, den man testen könnte.

Changelog allerdings nicht vorhanden. Ob man den hier braucht, ist diskussionswürdig, ich mache ihn trotzdem noch mal auf und verlange ein Changelog, weil den Satz zu schreiben schneller geht, als sich 10 Minuten darüber zu streiten.

Comment 3 Florian Best

2012-09-14 14:02:10 CEST

(In reply to comment #2)
Changelog hinzugefügt.

Comment 4 Dirk Wiesenthal

2012-09-14 14:55:25 CEST

Changelog vorhanden

Comment 5 Stefan Gohmann

2012-12-12 21:08:31 CET

UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

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