yahoo-eng-team team mailing list archive

[OpenStack Infra <2089223@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/c/openstack/horizon/+/935854
Committed: https://opendev.org/openstack/horizon/commit/5093c4101edaaceaa258f1586b5f2a99dc101f71
Submitter: "Zuul (22348)"
Branch:    master

commit 5093c4101edaaceaa258f1586b5f2a99dc101f71
Author: chung00-lee <chung00.lee@xxxxxxxxxxx>
Date:   Thu Nov 21 17:15:45 2024 +0900

    Replace placement request path
    
    Remove '//' in the placement request path.
    
    Closes-Bug: #2089223
    Change-Id: I2646ac3f82c7e1ab01faf1626bd8aa8046d66d0d


** Changed in: horizon
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/2089223

Title:
  Replace placement request path

Status in OpenStack Dashboard (Horizon):
  Fix Released

Bug description:
  If the placement endpoint contains '/' at the end of the path such as 'http://placement.com/', the placement api request results in 404 not found. 
  It is because there are duplicate '/' characters after calling the function '_get_json'.

  For example, When using function 'resource_privoders', If the endpoint
  is 'http://placement.com/', the API request is performed as
  'http://placement.com//resource_providers'.

  https://opendev.org/openstack/horizon/src/branch/master/openstack_dashboard/api/placement.py#L56

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/2089223/+subscriptions