Bug 56830 - univention-repository-create doesn't use correct scheme for repository/online/server
Summary: univention-repository-create doesn't use correct scheme for repository/online...
Status: REOPENED
Alias: None
Product: UCS
Classification: Unclassified
Component: UMC - Repository settings
Version: UCS 5.0
Hardware: Other Linux
: P5 minor
Target Milestone: UCS 5.2-0-errata
Assignee: UMC maintainers
QA Contact: UMC maintainers
URL: https://git.knut.univention.de/univen...
Keywords:
Depends on: 54495
Blocks:
  Show dependency tree / graph
 
Reported: 2023-11-16 15:32 CET by Finn David
Modified: 2025-04-03 13:14 CEST (History)
4 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 2: Will only affect a few installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.034
Enterprise Customer affected?: Yes
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2023102221000027
Bug group (optional):
Customer ID:
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 Finn David univentionstaff 2023-11-16 15:32:27 CET 
A customer used `univention-repository-create` to create a local repository.
He noticed the following error message:

"W: Value '<FQDN>' incompatible for 'repository/online/server', but setting anyway"



According to https://forge.univention.org/bugzilla/show_bug.cgi?id=55732#c0 :

"ucr type checking requires repository/online/server to be of type url_http. i.e. the scheme has to be http or https."

It looks like `univention-repository-create` is not using the scheme that ucr type checking requires.
Comment 1 Florian Best univentionstaff 2023-11-16 17:00:05 CET 
Thank you!
MR: https://git.knut.univention.de/univention/ucs/-/merge_requests/964
Comment 2 Felix Botner univentionstaff 2025-01-28 13:17:54 CET 
sort of half fixed by 

d9e06dbcf8a2ea2bea393a09c1d2bfe5b045f543
b4c22827d39e4bc59bbd17533438ba6370061093

but did not remove the deprecated ucs variables
Comment 3 Felix Botner univentionstaff 2025-04-03 13:14:02 CEST 
We experienced with the new http(s):// prefix as the repositoryServer does not allow that, but UCR requires it now.

So that has to be sorted out, additional see what else from the MR is missing.