Simple interface for setting app config.

Simple interface for setting app config.

This adds a bare-bones HTML form for configuring the application config
parameters, i.e. client ID, service account key, rietveld server URL and
Rietveld nickname for the bot.

R=iannucci@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216374

[Mattias Nissler (ping if slow), 2013-08-05 09:12:00]

Please review.

[Mattias Nissler (ping if slow), 2013-08-05 18:17:12]

cc agable

[Mattias Nissler (ping if slow), 2013-08-08 10:14:42]

Gentle ping - can one of you take a look?

[iannucci, 2013-08-08 10:37:24]

lgtm

https://codereview.chromium.org/20514002/diff/1/reviewbot/app_config.py
File reviewbot/app_config.py (right):

https://codereview.chromium.org/20514002/diff/1/reviewbot/app_config.py#newco...
reviewbot/app_config.py:47: if e[1].status != 302:
logging of some sort?

[Mattias Nissler (ping if slow), 2013-08-08 10:54:34]

https://codereview.chromium.org/20514002/diff/1/reviewbot/app_config.py
File reviewbot/app_config.py (right):

https://codereview.chromium.org/20514002/diff/1/reviewbot/app_config.py#newco...
reviewbot/app_config.py:47: if e[1].status != 302:
On 2013/08/08 10:37:25, iannucci wrote:
> logging of some sort?

The exception will just bubble up in the logs. I think that's good enough, in
particular given that this is a bare-bones request handler for one-time setup.

[Mattias Nissler (ping if slow), 2013-08-08 10:54:56]

Committed patchset #1 manually as r216374 (presubmit successful).