Project

Profile

Help

Task #6771

closed

Users not setting CONTENT_ORIGIN receive an error message that is not the most helpful

Added by bmbouter over 2 years ago. Updated about 2 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 75
Quarter:

Description

Background

The CONTENT_ORIGIN setting is a required Pulp setting, and if unset Pulp refuses to start. This is by design.

In that situation the use receives a message that says You must specify the CONTENT_ORIGIN setting. This comes from this line in the settings file: https://github.com/pulp/pulpcore/blob/master/pulpcore/app/settings.py#L233

This error message is technically correct, but it's not very helpful.

Solution

  1. Switch the exception type to be a Django ImproperlyConfigured exception.
  2. Make the text longer. It should include:
  • That the CONTENT_ORIGIN setting is required.
  • That the Installer sets it automatically
  • It could be a permissions problem.
Actions #1

Updated by fao89 over 2 years ago

  • Tracker changed from Issue to Task
  • % Done set to 0
  • Sprint set to Sprint 73
Actions #2

Updated by lmjachky over 2 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to lmjachky
Actions #3

Updated by pulpbot over 2 years ago

  • Status changed from ASSIGNED to POST
Actions #4

Updated by rchan over 2 years ago

  • Sprint changed from Sprint 73 to Sprint 74
Actions #5

Updated by rchan over 2 years ago

  • Sprint changed from Sprint 74 to Sprint 75

Added by Lubos Mjachky over 2 years ago

Revision d7163c32

Provide a more descriptive error message

closes #6771

Actions #6

Updated by Anonymous over 2 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #7

Updated by fao89 about 2 years ago

  • Sprint/Milestone set to 3.5.0
Actions #8

Updated by pulpbot about 2 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF