Bug 54769 - jpegPhoto syntax has no error message anymore but throws exception
jpegPhoto syntax has no error message anymore but throws exception
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UDM (Generic)
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0-1-errata
Assigned To: Florian Best
Johannes Keiser
: python3-migration
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2022-05-18 16:06 CEST by Florian Best
Modified: 2022-06-08 16:50 CEST (History)
0 users

See Also:
What kind of report is it?: Development Internal
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): Regression
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2022-05-18 16:06:42 CEST
git:2d1c7a160c0bc6628748ead5e0b4771a646ae788 added a `raise` statement to the `jpegPhoto` syntax class:

                                except (KeyError, IOError, IndexError):
                                        ud.debug(ud.ADMIN, ud.WARN, 'Failed to convert PNG file into JPEG: %s' % (traceback.format_exc(),))
+                                       raise
                                        raise univention.admin.uexceptions.valueError(_('Failed to convert PNG file into JPEG format.')
Comment 1 Florian Best univentionstaff 2022-05-18 17:02:01 CEST
The raise line has been removed again.

univention-directory-manager-modules.yaml
fb5e60b895ba | Bug #54769: fix exception handling of jpegPhoto syntax class

univention-directory-manager-modules (15.0.11-48)
fb5e60b895ba | Bug #54769: fix exception handling of jpegPhoto syntax class
Comment 2 Johannes Keiser univentionstaff 2022-05-25 11:06:29 CEST
OK: code changes
OK: yaml / package build
OK: Validation message is shown instead of traceback
-> verified