dreamingmind / bindery

CakePHP 1.3 Based Dreaming Mind Bindery Site
2 stars 2 forks source link

https forced redirect is not carrying all the url parameteres #261

Open dreamingmind opened 9 years ago

dreamingmind commented 9 years ago

In AppController->beforeFilter() there is a check to see if the user is in https and if the current action requires secure connections.

If need be, it will redirect to $this->force(), but in a case where there were query param ( ?name=value) the params were lost. Other things (like anchors) may get lost too.

Probably forces user of $this->here is not adequate when there is a complex address involved.