S3_token would error out when we have no path

Bug #956983 reported by Chmouel Boudjnah
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Undecided
Chmouel Boudjnah

Bug Description

stack@devstack:~$ curl http://localhost:8080
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/eventlet/wsgi.py", line 336, in handle_one_response
    result = self.application(self.environ, start_response)
  File "/opt/stack/swift/swift/common/middleware/healthcheck.py", line 38, in __call__
    return self.app(env, start_response)
  File "/opt/stack/swift/swift/common/middleware/memcache.py", line 47, in __call__
    return self.app(env, start_response)
  File "/opt/stack/swift/swift/common/middleware/swift3.py", line 461, in __call__
    return self.app(env, start_response)
  File "/opt/stack/keystone/keystone/middleware/s3_token.py", line 58, in __call__
    parts = swift_utils.split_path(req.path, 1, 4, True)
  File "/opt/stack/swift/swift/common/utils.py", line 232, in split_path
    raise ValueError('Invalid path: %s' % quote(path))
ValueError: Invalid path: /

Changed in keystone:
assignee: nobody → Chmouel Boudjnah (chmouel)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

Fix proposed to branch: master
Review: https://review.openstack.org/5453

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/5453
Committed: http://github.com/openstack/keystone/commit/7abe0aa3845459b95a7d4e401e51d4ab8c4c0280
Submitter: Jenkins
Branch: master

commit 7abe0aa3845459b95a7d4e401e51d4ab8c4c0280
Author: Chmouel Boudjnah <email address hidden>
Date: Wed Mar 21 16:59:15 2012 +0000

    S3 tokens cleanups.

    - Cleanups.
    - Remove reference about config admin_username/password/token.
    - Return proper http error on errors.
    - Add unittests (skip them for now when swift is not installed).
    - Fixes bug 956983.

    Change-Id: I392fc274f3b01a5a0b5779dd13f9cd3b819ee65a

Changed in keystone:
status: In Progress → Fix Committed
Joseph Heck (heckj)
Changed in keystone:
milestone: none → essex-rc1
Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: essex-rc1 → 2012.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.