Project

Profile

Help

Issue #3476

Base path validation on distributions doesn't work depending on your hostname

Added by daviddavis over 1 year ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 34

Description

@ichimonji10 discovered that perfectly valid hostnames weren't being accepted after this PR got merged:

https://github.com/pulp/pulp/pull/3371

We're using django's URLValidator which apparently looks for a dot in the hostname:

https://github.com/django/django/blob/master/django/core/validators.py#L86

That is except if you are using localhost:

https://github.com/django/django/blob/master/django/core/validators.py#L93

Associated revisions

Revision 855d2025 View on GitHub
Added by daviddavis over 1 year ago

Fixing problems with bad hostnames

Looks like django requires certain features of hostnames like a "." so
using `get_host()` was not a good idea. Using a known valid hostname
instead.

fixes #3476
https://pulp.plan.io/issues/3476

Revision 855d2025 View on GitHub
Added by daviddavis over 1 year ago

Fixing problems with bad hostnames

Looks like django requires certain features of hostnames like a "." so
using `get_host()` was not a good idea. Using a known valid hostname
instead.

fixes #3476
https://pulp.plan.io/issues/3476

Revision 855d2025 View on GitHub
Added by daviddavis over 1 year ago

Fixing problems with bad hostnames

Looks like django requires certain features of hostnames like a "." so
using `get_host()` was not a good idea. Using a known valid hostname
instead.

fixes #3476
https://pulp.plan.io/issues/3476

History

#1 Updated by daviddavis over 1 year ago

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

#2 Updated by daviddavis over 1 year ago

  • Tags Pulp 3, Pulp 3 MVP added

#3 Updated by dalley over 1 year ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 34

#4 Updated by daviddavis over 1 year ago

  • Status changed from POST to MODIFIED

#5 Updated by dkliban@redhat.com over 1 year ago

  • Sprint/Milestone set to 3.0

#6 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3, Pulp 3 MVP)

Please register to edit this issue

Also available in: Atom PDF