Add /auth/api/v1/server/info endpoints.

Add /auth/api/v1/server/info endpoints.

Serve it and fetch it. For interoperability with luci-py components.auth.

Will be used by luci-cron when configuring PubSub topic to pass to Swarming.

R=maruel@chromium.org
BUG=

Committed: https://github.com/luci/luci-go/commit/d7afe013e8d7c0ab5b8fdeebc7f537ef6f9b6e9a

[Vadim Sh., 2015-11-24 03:31:28]

e.g. https://luci-cron.appspot.com/auth/api/v1/server/info

also compare to https://chromium-swarm.appspot.com/auth/api/v1/server/info

[Vadim Sh., 2015-11-24 03:32:19]

Description was changed from

==========
Add /auth/api/v1/server/info endpoints.

Server it and fetch it. For interoperability with luci-py components.auth.

Will be used by luci-cron when configuring PubSub topic to pass to Swarming.

R=maruel@chromium.org
BUG=
==========

to

==========
Add /auth/api/v1/server/info endpoints.

Serve it and fetch it. For interoperability with luci-py components.auth.

Will be used by luci-cron when configuring PubSub topic to pass to Swarming.

R=maruel@chromium.org
BUG=
==========

[M-A Ruel, 2015-11-24 13:48:00]

https://codereview.chromium.org/1468053004/diff/20001/appengine/gaeauth/serve...
File appengine/gaeauth/server/default.go (right):

https://codereview.chromium.org/1468053004/diff/20001/appengine/gaeauth/serve...
appengine/gaeauth/server/default.go:63: AppRuntime:         "go",
Do you think it's useful to have runtime version? E.g.

RuntimeVersion: runtime.Version(),

https://codereview.chromium.org/1468053004/diff/20001/server/auth/info/info.go
File server/auth/info/info.go (right):

https://codereview.chromium.org/1468053004/diff/20001/server/auth/info/info.g...
server/auth/info/info.go:34: // FetchServiceInfo fetches information about the
service at given URL.
Err, why not just do a HTTP 301 ?

[Vadim Sh., 2015-11-24 19:27:29]

https://codereview.chromium.org/1468053004/diff/20001/appengine/gaeauth/serve...
File appengine/gaeauth/server/default.go (right):

https://codereview.chromium.org/1468053004/diff/20001/appengine/gaeauth/serve...
appengine/gaeauth/server/default.go:63: AppRuntime:         "go",
On 2015/11/24 13:47:59, M-A Ruel wrote:
> Do you think it's useful to have runtime version? E.g.
> 
> RuntimeVersion: runtime.Version(),

Done.

https://codereview.chromium.org/1468053004/diff/20001/server/auth/info/info.go
File server/auth/info/info.go (right):

https://codereview.chromium.org/1468053004/diff/20001/server/auth/info/info.g...
server/auth/info/info.go:34: // FetchServiceInfo fetches information about the
service at given URL.
On 2015/11/24 13:47:59, M-A Ruel wrote:
> Err, why not just do a HTTP 301 ?

What? Why? I don't understand.

This function will be called like FetchServiceInfo(c,
"https://chromium-swarm-dev.appspot.com") to get service account name of
swarming. I don't like hardcoding some arbitrary rules for hostname => service
account name conversion (since they have exceptions e.g. googleplex apps).

[M-A Ruel, 2015-11-24 19:31:34]

https://codereview.chromium.org/1468053004/diff/20001/server/auth/info/info.go
File server/auth/info/info.go (right):

https://codereview.chromium.org/1468053004/diff/20001/server/auth/info/info.g...
server/auth/info/info.go:34: // FetchServiceInfo fetches information about the
service at given URL.
On 2015/11/24 19:27:29, Vadim Sh. wrote:
> On 2015/11/24 13:47:59, M-A Ruel wrote:
> > Err, why not just do a HTTP 301 ?
> 
> What? Why? I don't understand.
> 
> This function will be called like FetchServiceInfo(c,
> "https://chromium-swarm-dev.appspot.com") to get service account name of
> swarming. I don't like hardcoding some arbitrary rules for hostname => service
> account name conversion (since they have exceptions e.g. googleplex apps).

I mean, chromium-swarm-dev could redirect this API call to chrome-auth-dev? I
guess I miss something obvious here.

[Vadim Sh., 2015-11-24 19:35:02]

On 2015/11/24 19:31:34, M-A Ruel wrote:
> https://codereview.chromium.org/1468053004/diff/20001/server/auth/info/info.go
> File server/auth/info/info.go (right):
> 
>
https://codereview.chromium.org/1468053004/diff/20001/server/auth/info/info.g...
> server/auth/info/info.go:34: // FetchServiceInfo fetches information about the
> service at given URL.
> On 2015/11/24 19:27:29, Vadim Sh. wrote:
> > On 2015/11/24 13:47:59, M-A Ruel wrote:
> > > Err, why not just do a HTTP 301 ?
> > 
> > What? Why? I don't understand.
> > 
> > This function will be called like FetchServiceInfo(c,
> > "https://chromium-swarm-dev.appspot.com") to get service account name of
> > swarming. I don't like hardcoding some arbitrary rules for hostname =>
service
> > account name conversion (since they have exceptions e.g. googleplex apps).
> 
> I mean, chromium-swarm-dev could redirect this API call to chrome-auth-dev? I
> guess I miss something obvious here.

auth_service is not involved here in any way.
https://<server>/auth/api/v1/server/info returns information about the <server>,
in particular authentication related info (application identity). Also it is
complimentary to https://<server>/auth/api/v1/server/certificates: the
application certificates encode same identity information (service account name
is embedded in the certificate too (and signed by Google CA)).

[M-A Ruel, 2015-11-24 19:42:07]

lgtm

[Vadim Sh., 2015-11-24 19:42:43]

The CQ bit was checked by vadimsh@chromium.org

[commit-bot: I haz the power, 2015-11-24 19:42:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1468053004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1468053004/40001

[commit-bot: I haz the power, 2015-11-24 19:45:24]

Committed patchset #3 (id:40001) as
https://github.com/luci/luci-go/commit/d7afe013e8d7c0ab5b8fdeebc7f537ef6f9b6e9a