Bug 56551 – Wrong attribute name used for detecting if mail is present mandatory attributes

Bug 56551 - Wrong attribute name used for detecting if mail is present mandatory attributes


Summary:	Wrong attribute name used for detecting if mail is present mandatory attributes

Status:	NEW

Product:	UCS@school
Classification:	Unclassified
Component:	Import scripts
Version:	UCS@school 5.0
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	UCS@school maintainers
QA Contact:

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2023-09-07 10:05 CEST by Jan-Luca Kiok
Modified:	2023-09-07 12:22 CEST (History)
CC List:	1 user (show)

See Also:
What kind of report is it?:	Bug Report
What type of bug is this?:	5: Major Usability: Impairs usability in key scenarios
Who will be affected by this bug?:	1: Will affect a very few installed domains
How will those affected feel about the bug?:	1: Nuisance – not a big deal but noticeable
User Pain:	0.029
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):	bitesize, Error handling
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 Jan-Luca Kiok

2023-09-07 10:05:29 CEST

It was reported that in ucs-school-import/modules/ucsschool/importer/models/import_user.py 'mailPrimaryAttribute' is used when it should be 'mailPrimaryAddress' instead:


859                 except (AttributeError, IndexError):
860                     if (
861                         "email" in self.config["mandatory_attributes"]
862                         or "mailPrimaryAttribute" in self.config["mandatory_attributes"]
863                     ):

and

877             except EmptyFormatResultError:
878                 if (
879                     "email" in self.config["mandatory_attributes"]
880                     or "mailPrimaryAttribute" in self.config["mandatory_attributes"]
881                 ):