Project

General

Profile

Bug #1779

rgw: swift auth returns wrong error code when unexisting user is given

Added by Yehuda Sadeh over 12 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
Low
Assignee:
-
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

returns 404 instead of 403

Associated revisions

Revision 4089001d (diff)
Added by Christophe Courtaut over 10 years ago

rgw: Fix return value for swift user not found

http://tracker.ceph.com/issues/1779 fixes #1779

Adjust the return value from rgw_get_user_info_by_swift call
in RGW_SWIFT_Auth_Get::execute() to have the correct
return code in response.

Revision 54ee2dc8 (diff)
Added by Christophe Courtaut over 10 years ago

rgw: Fix return value for swift user not found

http://tracker.ceph.com/issues/1779 fixes #1779

Adjust the return value from rgw_get_user_info_by_swift call
in RGW_SWIFT_Auth_Get::execute() to have the correct
return code in response.
(cherry picked from commit 4089001de1f22d6acd0b9f09996b71c716235551)

History

#1 Updated by Sage Weil over 12 years ago

  • translation missing: en.field_position set to 1
  • translation missing: en.field_position changed from 1 to 1030

#2 Updated by Sage Weil over 12 years ago

  • translation missing: en.field_position deleted (1030)
  • translation missing: en.field_position set to 52

#3 Updated by Sage Weil about 12 years ago

  • Category set to 22

#4 Updated by Sage Weil over 11 years ago

  • Project changed from Ceph to rgw
  • Category deleted (22)

#5 Updated by Ian Colle about 11 years ago

  • Priority changed from Normal to Low

#6 Updated by Ian Colle about 11 years ago

Work in course with other swift changes, but not a driver.

#7 Updated by Sage Weil about 11 years ago

  • Status changed from New to 12

#8 Updated by Christophe Courtaut over 10 years ago

Here comes a fix for this issue.

https://github.com/ceph/ceph/pull/385

#9 Updated by Christophe Courtaut over 10 years ago

This issue should be closed as https://github.com/ceph/ceph/pull/385 was merged.

#10 Updated by Greg Farnum over 10 years ago

  • Status changed from 12 to Pending Backport

Do we want to backport this anywhere?

#11 Updated by Christophe Courtaut over 10 years ago

I need some advice here, if it need some backport, and if so where?

#12 Updated by Christophe Courtaut over 10 years ago

As yehuda told me on the mailing list, the fix might not be backported.

Should we close the issue?

#13 Updated by Sage Weil over 10 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF