Code review comment for lp:~mwhudson/launchpad/vostok-traverse-distro

Revision history for this message
Tim Penhey (thumper) wrote :

Agh - perhaps it is just me, but __get_item__ on I<any>Set just seems wrong to
me.

> return self.redirectSubTree(canonical_url(distro), status=301)

That will take the user out of the vostok vhost. Is that what you want?

If not, you probably want to set the root as part of the canonical_url call.

review: Needs Information

« Back to merge proposal