-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
Mend Scan Summary: ❌Repository: open-component-model/mpas-product-controller
|
u, err := v.parseDomainURL(repository.Spec.Domain, repository.Spec.Insecure) | ||
if err != nil { | ||
return false, fmt.Errorf("failed to parse repository url %s: %w", repository.Spec.Domain, err) | ||
} | ||
|
||
domain = u |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this logic go in API as kubebuilder annotation?
+kubebuilder:validation:Pattern="^(http|https)://.*$"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, because we specifically disabled that on the repository.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't allow passing in the scheme to the repository because of other bugs that it did.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok 👍🏻
u, err := v.parseDomainURL(repository.Spec.Domain, repository.Spec.Insecure) | ||
if err != nil { | ||
return false, fmt.Errorf("failed to parse repository url %s: %w", repository.Spec.Domain, err) | ||
} | ||
|
||
domain = u |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok 👍🏻
Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
What type of PR is this? (check all applicable)
Related Tickets & Documents
Screenshots
Added tests?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Added to documentation?
Checklist: