Make Web UI URL configurable

[wereHamster]

The Web UI URL (/transmission/web) is currently hardcoded in several places (C, JavaScript code). This makes it impossible to configure a http-server as a proxy for more then one transmission daemon (at least not without using some kind of reverse-proxy that rewrites the URLs in the response). It would be nice to make the Web URL configurable.

[charles]

I'd like to see #1559 get checked in first.

[charles]

wereHamster: could you sync this patch with trunk?

[wereHamster]

I changed the default to /, I think it makes more sense then /transmission/web - the web server runs on a separate port anyway. The rpc url is /rpc, so any scripts would need to be updated. If you don't like it, you can change the default in session.c. Just make sure it ends with a / (eg. /transmission/web/).

[charles]

05:10 < wereHamster> charles_: I won't have time in the next week to work on T (I _really_ should read the papers at least once before I turn up at the exams)
05:11 < wereHamster> so maybe you should to postpone 1538 until 1.60

[charles]

wereHamster: reopen this ticket when you think the patch is ready. I agree with the premise and will use the finished patch into svn.

[wereHamster]

I updated the patches. The first one only changes javascript files, the second one contains the remaining changes. Unlike I said in comment 5, the default url is /transmission/ (thus won't break any existing scripts). Also, the handling of URLs changed slightly. If the url is .../rpc, it's handled by handle_rpc(), if the url is .../upload, it's handled by handle_upload(), else the url is handled by handle_clutch(). So there's no .../web url anymore. The downside of that is that it can't serve any files that start with 'rpc' or 'upload', but I don't think it's a big issue.

[charles]

wereHamster: somehow I didn't notice that this ticket was reopened. If the patches are still good, let me know and I'll put them into trunk.

[wereHamster]

This is both former patches merged into a single one

[charles]

This ticket just keeps falling through the cracks.

wereHamster: is that latest revision ready for commiting?

[charles]

This ticket just keeps falling through the cracks.

wereHamster: if this patch is ready please set the milestone @ 1.90

[wereHamster]

I'll wait for you to commit a certain patch because it conflicts with my changes. Once you do so I'll update my two patchs and attach them here.

[wereHamster]

github compare view: ​http://github.com/wereHamster/transmission/compare/master...ticket/1538-configurable-web-url

[niol]

updated patch

[niol]

I've been running transmission with this patch for a week now, and it works great. Please let me know if I need to improve something for it to be merged.

[niol]

transmission-remote cli utility corresponding update

[niol]

I added the corresponding enhancement to the transmission-remote tool. I've made the change in a backward compatible way (i.e. command line with only host:port or host or port still work). Please tell me if I need to further fix things.

[wereHamster]

ticket-1538-configurable-web-url.patch is freshly rebased on top of r11506. It contains all the needed changes (javascript, rpc-server and the remote).

[charles]

added to trunk by r11524