Warning: Can't synchronize with repository "(default)" (Unsupported version control system "svn": No module named svn). Look in the Trac log for more information.

Ticket #889 (closed defect: fixed)

Opened 13 years ago

Last modified 12 years ago

[PATCH] url('foo') unexpected behaviour (server.webpath prepended)

Reported by: anonymous Owned by: anonymous
Priority: normal Milestone:
Component: TurboGears Version: 0.9a6
Severity: major Keywords:
Cc:

Description

Using url('foo') when server.webpath is set yields unexpected (in my opinion) result of server.webpath + 'foo'. It would be more natural for url() to return 'foo', or full path with components between app root and current "node" included.

Current behaviour can really break things, e.g. after logging in quickstart application (not mounted on web server's "/"), where browser is forwarded to a location like:  http://foo.bar/baz/http://foo.bar/baz/page

Attachments

tg-url-fix-relative.patch Download (658 bytes) - added by jkoszuk-at-poee-dot-eu-dot-org 13 years ago.

Change History

Changed 13 years ago by jkoszuk-at-poee-dot-eu-dot-org

comment:1 Changed 13 years ago by roger.demetrescu

  • Summary changed from url('foo') unexpected behaviour (server.webpath prepended) to [PATCH] url('foo') unexpected behaviour (server.webpath prepended)

make it visible in "Pending Patches" report

comment:2 Changed 13 years ago by kevin

  • Status changed from new to closed
  • Resolution set to fixed

committed in [1626]. thanks!

Note: See TracTickets for help on using tickets.