Project

Profile

Help

Issue #8957

closed

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

Added by sajha over 3 years ago. Updated over 3 years 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.7CLOSED - DUPLICATE

Actions
Actions #1

Updated by daviddavis over 3 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to daviddavis
Actions #2

Updated by daviddavis over 3 years ago

Actions #3

Updated by pulpbot over 3 years ago

  • Status changed from ASSIGNED to POST

Added by daviddavis over 3 years ago

Revision c5f56257 | View on GitHub

Fix error message when invalid requirements url

fixes #8957

Actions #4

Updated by daviddavis over 3 years ago

  • Status changed from POST to MODIFIED
Actions #6

Updated by fao89 over 3 years ago

  • Sprint/Milestone set to 0.8.1

Added by daviddavis over 3 years ago

Revision 63c014f1 | View on GitHub

Fix error message when invalid requirements url

fixes #8957

(cherry picked from commit c5f5625749a03ef6d869f37ad64574cff6b89358)

Actions #7

Updated by pulpbot over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF