Refactor of LibraryFileAlias.[secure_]url
Bug #76743 reported by
Guilherme Salgado
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Medium
|
Guilherme Salgado |
Bug Description
Currently, we have LibraryFileAlia
I discussed this with Steve and the plan is to define http_url (identical to the existing url) and https_url (identical to the current securel_url) properties and a getURL() method, where the latter will check the value of config.
Changed in launchpad: | |
assignee: | nobody → salgado |
importance: | Undecided → Medium |
status: | Unconfirmed → Confirmed |
Changed in launchpad: | |
status: | Confirmed → In Progress |
Changed in launchpad: | |
status: | In Progress → Fix Committed |
Changed in launchpad: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Guilherme Salgado wrote:
> Currently, we have LibraryFileAlia s.url and LibraryFileAlia s.secure_ url,
> where the latter just replace 'http' with 'https' from the former's
> value and returns it. This is a problem because we don't want to have
> any https links on our development instance.
It is because while most stuff we want retrieved from the Librarian via
HTTP, images that get embedded in Launchpad pages such as emblems and
hackergotchis need to be retrieved via HTTPS to avoid warnings on some
browsers. So where this is the case, .secure_url is used to calculate the
URL rather than just .url
> I discussed this with Steve and the plan is to define http_url launchpad. virtual_ hosts.user_ https and return
> (identical to the existing url) and https_url (identical to the current
> securel_url) properties and a getURL() method, where the latter will
> check the value of config.
> either the value of http_url or https_url.
This bug report does not explain why any change is necessary - it just gives
a solution without explaining the problem.
-- www.canonical. com/ www.ubuntu. com/
Stuart Bishop <email address hidden> http://
Canonical Ltd. http://