New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 754521 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 730832



Sign in to add a comment

luci-config.appspot.com is hardcoded in API Explorer link

Project Member Reported by no...@chromium.org, Aug 11 2017

Issue description

problem:

API Explorer link in
https://a-dot-luci-config.appspot.com
points to
https://luci-config.appspot.com/_ah/api/explorer

instead of

/_ah/api/explorer

please use the relative url
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/luci/luci-py.git/+/2b6b39ca19426b32d0f3e911d7584fd3cbe2e0f4

commit 2b6b39ca19426b32d0f3e911d7584fd3cbe2e0f4
Author: Cameron Payton <cwpayton@google.com>
Date: Fri Aug 11 22:04:38 2017

config_service: Changed fixed URL to relative URL in config-ui.html

APIs Explorer was fixed at https://luci-config.appspot.com/_ah/api/explorer
but is now relative to instance

Bug:  754521 
Change-Id: I2514867de3c8bfb149cbf97e4979d157f73cf93e
Reviewed-on: https://chromium-review.googlesource.com/612461
Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: Ryan Tseng <hinoka@chromium.org>
Commit-Queue: Cameron Payton <cwpayton@google.com>

[modify] https://crrev.com/2b6b39ca19426b32d0f3e911d7584fd3cbe2e0f4/appengine/config_service/ui/README.md
[modify] https://crrev.com/2b6b39ca19426b32d0f3e911d7584fd3cbe2e0f4/appengine/config_service/ui/src/config-ui/config-ui.html

Owner: cwpayton@google.com
Status: Fixed (was: Untriaged)
Seems fixed by Cameron now :)

Sign in to add a comment