alfonsodg / demo-web2py

Apache License 2.0
0 stars 0 forks source link

Commit e6b7b7346e breaks everything(!) #172

Closed alfonsodg closed 10 years ago

alfonsodg commented 10 years ago

From gr...@jingojango.net on February 04, 2011 17:42:17

Most browsers seem to append '/' to requests - tested with Chrome and curl. Due to the regex change in commit e6b7b7346e on line 426 of rewrite.py, the following matches fail and return HTTP 400 codes (check on lines 472-476):

regex_url.match('/') regex_url.match('') <_sre.SRE_Match object at 0x19b9a40> regex_url.match('/admin/') regex_url.match('/admin') <_sre.SRE_Match object at 0x19b9ad0>

Reverting the regex to end with /?$) on line 426 fixes this.

Original issue: http://code.google.com/p/web2py/issues/detail?id=174

alfonsodg commented 10 years ago

From massimo....@gmail.com on February 05, 2011 22:25:03

Thanks Jonathan for the fix.

Status: Fixed