Bug 55573 - UCR module should give an error message if the UCRV is not to be saved due to invalid type
UCR module should give an error message if the UCRV is not to be saved due to...
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Univention Configuration Registry
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0-2-errata
Assigned To: Iván.Delgado
Peter Stoll
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2023-01-16 13:35 CET by Dirk Wiesenthal
Modified: 2023-02-28 17:06 CET (History)
1 user (show)

See Also:
What kind of report is it?: Feature Request
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: 2023022621000321
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Wiesenthal univentionstaff 2023-01-16 13:35:39 CET
When a UCRV has a type and the value that is to be set via the UMC module violates that, we need an error message for the user.

Also, would it be possible to include "Type: ..." into the output of "ucr info ..."?
Comment 2 Peter Stoll univentionstaff 2023-01-19 14:49:53 CET
QA Tasks:
- Code review
- Install packets incl. unit test
- Testing of ucr info with all combinations of type and extra parameters like max, min, regex etc.
- Test of UMD Module UCR genration error messages due to violating type contraints.
Comment 3 Iván.Delgado univentionstaff 2023-01-19 15:02:38 CET
The UCR type check is integrated in the UMC module and  info regarding the UCR type is visible in `ucr info` command

univention-management-console-module-ucr.yaml
663144be3a35 | Bug #55573: univention-management-console-module-ucr 9.0.0-9A~5.0.0.202301191336
acfa4fed14a4 | Bug #55573: update advisory and changelog

univention-management-console-module-ucr (9.0.0-9)
acfa4fed14a4 | Bug #55573: update advisory and changelog

univention-management-console-module-ucr (9.0.0-8)
f8e0557e028f | Bug #55573: modify handler_set to return validation type check and integrate it in umc

univention-config-registry.yaml
b8314e9ac8e2 | Bug #55573: univention-config-registry 15.0.7-23A~5.0.0.202301191345
acfa4fed14a4 | Bug #55573: update advisory and changelog

univention-config-registry (15.0.7-23)
acfa4fed14a4 | Bug #55573: update advisory and changelog

univention-config-registry (15.0.7-22)
e701175eeee1 | Bug #55573: Update ucr info to show UCR type
f8e0557e028f | Bug #55573: modify handler_set to return validation type check and integrate it in umc

ucs-test (5.0.44-1)
r55573 | Bug #27653: improve efficiency
Comment 5 Mirac Erdemiroglu univentionstaff 2023-02-28 17:06:27 CET
Customer effected Ticket#2023022621000321

## Check failed: 65_check_repository_config - Check repository configuration for correctness ##
Some UCR variables for the repository configuration are deprecated.
As these variables should be no longer used, this check has been added to check for the existence of these variables.
Use the {appcenter:components} to correct these values by once saving the General repository settings 
as well as saving the settings for all Additional repositories or press the Button ADJUST ALL COMPONENTS 
to correct these settings and delete the obsolete variables.
Furthermore, it is checked if the scheme of the server variable is either http or https
The variable 'repository/online/component/backend7106/username' is deprecated and should no longer be used.
The variable 'repository/online/component/frontend7106/username' is deprecated and should no longer be used.
The variable 'repository/online/component/mobileapi1143/username' is deprecated and should no longer be used.
The variable 'repository/online/component/mobileapirestricted211/username' is deprecated and should no longer be used.
The variable 'repository/online/component/oxdocconv7106/username' is deprecated and should no longer be used.