Standard way of finding mugshot url, default if not set

Bug #340640 reported by Dave Walker
16
Affects Status Importance Assigned to Milestone
Launchpad itself
Won't Fix
Low
Unassigned

Bug Description

Using either the API or a http(s) url to access a mugshot is currently difficult.

The api mugshot_link returns something like:
https://api.launchpad.net/beta/~davewalker/mugshot
(which isn't that useful)

Using the open api:
https://launchpad.net/api/beta/~davewalker/mugshot -- returns an ugly error
https://launchpad.net/api/beta/~ubuntu-uk/mugshot -- redirects to the real image

Would be good if there was a way of attempting to get the user mugshot url, and if not set return the default (https://launchpad.net/@@/person-mugshot)

The launchpad code base must do something similar to show the user profile.

Revision history for this message
Curtis Hovey (sinzui) wrote :

This looks like a case where the web intent is incompatible with the API intent.

Changed in launchpad-registry:
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Dave Walker (davewalker) wrote :

Is there an update on this bug?

Revision history for this message
Curtis Hovey (sinzui) wrote :

We have no plans to work on this bug at the moment. The error the API is returning is really the same error at the model level. Launchpad uses a formatter when generating pages to show the generic image when one is selected.

Launchpad is going into a UI development phase where we will review the use of all icons, logos, and mugshots to make all the rules consistent.

Curtis Hovey (sinzui)
tags: added: api
description: updated
description: updated
Revision history for this message
Robert Collins (lifeless) wrote :

So, I think the underlying issue here is a confused website; AIUI this is for the loco directory, which has been embedding lp API urls in *its* output: this is a bit crazy, and terribly slow (we issue a 302 for /every/ mugshot etc).

I think its better for us to fix the API on person / team to emit the logo and mugshot references directly - they will more reliably be None then when unset.

Changed in launchpad:
status: Triaged → Won't Fix
Revision history for this message
Robert Collins (lifeless) wrote :

See bug 713873 which I filed to address the use case for loco directory.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.