Checkout doesn't work if you create an url alias of 'cart/checkout'.

The reason is that 'uc_referer_check' doesn't check url aliases.

I've made a patch to correct this, but I'm not sure if it breaks any security lock, I hope not.

CommentFileSizeAuthor
uc_store.module.patch696 bytesjherencia
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rszrama’s picture

Status: Needs review » Postponed

I generally don't recommend aliasing the checkout URLs, as who knows what else is depending on them. At this point, I'd postpone this patch as it's not something I want to have to go through the full debug process for this late in the development cycle of the 2.0.

Andy_Lowe’s picture

Version: 6.x-2.0-rc3 » 7.x-3.x-dev
Category: support » feature
Status: Postponed » Closed (won't fix)

My feeling is that the benefits of this feature do not out way the risk of introducing a security vulnerability in the checkout process.