auth: Add a page to setup web client ID for a service.

auth: Add a page to setup web client ID for a service.

Note that LUCI config is undesirable here. A web client ID will be needed for
initial setup of Auth Service, but LUCI Config requires a working auth service.
If we make Auth Service setup depend on LUCI Config, we'll have a dependency
cycle.

R=nodir@chromium.org
BUG=714821
Review-Url: https://codereview.chromium.org/2836223002
Committed: https://github.com/luci/luci-py/commit/da04014d32632924f7256b27fa0235b727161e23

[Vadim Sh., 2017-04-24 22:26:06]

PTAL

Example page:
https://2768-d76c2a4-tainted-vadimsh-dot-chrome-infra-auth-dev.appspot.com/au...

[nodir, 2017-04-24 22:34:24]

lgtm

https://codereview.chromium.org/2836223002/diff/20001/appengine/components/co...
File appengine/components/components/auth/api.py (right):

https://codereview.chromium.org/2836223002/diff/20001/appengine/components/co...
appengine/components/components/auth/api.py:863: if web_id:
and not in additional_client_ids

[Vadim Sh., 2017-04-24 22:40:23]

https://codereview.chromium.org/2836223002/diff/20001/appengine/components/co...
File appengine/components/components/auth/api.py (right):

https://codereview.chromium.org/2836223002/diff/20001/appengine/components/co...
appengine/components/components/auth/api.py:863: if web_id:
On 2017/04/24 22:34:23, nodir wrote:
> and not in additional_client_ids

This is unnecessary, deduplication happens later in AuthDB constructor where all
sources of client IDs are combined together.

[Vadim Sh., 2017-04-24 22:40:31]

The CQ bit was checked by vadimsh@chromium.org

[commit-bot: I haz the power, 2017-04-24 22:40:35]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-24 22:43:06]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493073631265330,
"parent_rev": "d76c2a46ff00529daa5abe668dd99c7bc1bfe3f0", "commit_rev":
"da04014d32632924f7256b27fa0235b727161e23"}

[commit-bot: I haz the power, 2017-04-24 22:43:09]

Description was changed from

==========
auth: Add a page to setup web client ID for a service.

Note that LUCI config is undesirable here. A web client ID will be needed for
initial setup of Auth Service, but LUCI Config requires a working auth service.
If we make Auth Service setup depend on LUCI Config, we'll have a dependency
cycle.

R=nodir@chromium.org
BUG=714821
==========

to

==========
auth: Add a page to setup web client ID for a service.

Note that LUCI config is undesirable here. A web client ID will be needed for
initial setup of Auth Service, but LUCI Config requires a working auth service.
If we make Auth Service setup depend on LUCI Config, we'll have a dependency
cycle.

R=nodir@chromium.org
BUG=714821

Review-Url: https://codereview.chromium.org/2836223002
Committed:
https://github.com/luci/luci-py/commit/da04014d32632924f7256b27fa0235b727161e23
==========

[commit-bot: I haz the power, 2017-04-24 22:43:10]

Committed patchset #2 (id:20001) as
https://github.com/luci/luci-py/commit/da04014d32632924f7256b27fa0235b727161e23