Project

Profile

Help

Issue #8957

URL validation error on remote displays remote base URL instead of source url from requirements.

Added by sajha 3 months ago. Updated 2 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Katello
Sprint:
Quarter:

Description

The url validation is somehow leading to this: ["Invalid URL https://galaxy.ansible.com/. Ensure the URL ends '/'."]} This happens when I have some source specified in the requirements.yml without the trailing "/".. However the error displays the remote URL in the error which happens to have the trailing /..

(02:14:32 PM) rochacbruno: error is on https://github.com/pulp/pulp_ansible/blob/master/pulp_ansible/app/serializers.py#L166 (02:14:44 PM) rochacbruno: should be url=url instead of data["url"]


Related issues

Related to Ansible Plugin - Backport #8958: Backport 8957 to 0.7NEW

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision c5f56257 View on GitHub
Added by daviddavis 3 months ago

Fix error message when invalid requirements url

fixes #8957

Revision 63c014f1 View on GitHub
Added by daviddavis 2 months ago

Fix error message when invalid requirements url

fixes #8957

(cherry picked from commit c5f5625749a03ef6d869f37ad64574cff6b89358)

History

#1 Updated by daviddavis 3 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to daviddavis

#2 Updated by daviddavis 3 months ago

#3 Updated by pulpbot 3 months ago

  • Status changed from ASSIGNED to POST

#4 Updated by daviddavis 3 months ago

  • Status changed from POST to MODIFIED

#6 Updated by fao89 2 months ago

  • Sprint/Milestone set to 0.8.1

#7 Updated by pulpbot 2 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF