invalid URLs are not suppressed When use openstack endpoint create

Bug #1473639 reported by jiaxi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
In Progress
Medium
jiaxi

Bug Description

When creating endpoint with the command line below:
openstack --debug endpoint create nova public " http://127.0.0.1:8774 /v1.1/\$(tenant_i d)s"

Create endpoint successfully with invalid url.Obviously It should be suppressed as bug #1471034 said. it's implemented by the Identity v3.0 API.

This bug is different from bug #1471034, because bug #1471034 produced by keystone endpoint-create.It's implemented by
the Identity v2.0 API.

The Keystone command line interface packaged in python-keystoneclient only supports the Identity v2.0 API.
The OpenStack common command line interface packaged in python-openstackclient supports both v2.0 and v3 APIs.

So I modify the Identity v2.0 API can resolve bug #1471034. Review: https://review.openstack.org/#/c/200512/

I want to solved the url validation problem with two seperate git review.
Because this can be more effectively and easier to be done.

jiaxi (tjxiter)
Changed in keystone:
assignee: nobody → jiaxi (tjxiter)
jiaxi (tjxiter)
Changed in keystone:
status: New → In Progress
Changed in keystone:
importance: Undecided → Medium
Revision history for this message
David Stanek (dstanek) wrote :

I think this is actually a duplicate. Both changes describe the same broken server behavior.

Revision history for this message
jiaxi (tjxiter) wrote :

@David Stanek (dstanek):
Could I modify Identity V3 api in another commit?

Revision history for this message
David Stanek (dstanek) wrote :

> Could I modify Identity V3 api in another commit?

Sure, but I think you can probably just use the same commit since the validation logic should be shared.

Also, the openstack client can use v2.0. It's actually just a wrapper about keystone client.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers