<feed xmlns='http://www.w3.org/2005/Atom'>
<title>gluster-swift.git/test/unit/common/middleware, branch sp</title>
<subtitle></subtitle>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/'/>
<entry>
<title>Rebase swiftkerbauth imported code with upstream</title>
<updated>2014-01-28T06:55:55+00:00</updated>
<author>
<name>Chetan Risbud</name>
<email>crisbud@redhat.com</email>
</author>
<published>2014-01-23T07:12:53+00:00</published>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/commit/?id=03128e172e0242eba396e8487bdd8d6b0da52db3'/>
<id>03128e172e0242eba396e8487bdd8d6b0da52db3</id>
<content type='text'>
Few changes have been merged to upstream swiftkerbauth repo.
This commit brings it down to gluster-swift repo.
Bringing below changes to gluster-swift repo in one go.

http://review.gluster.org/#/c/6296/
http://review.gluster.org/#/c/6370/
http://review.gluster.org/#/c/6595/
http://review.gluster.org/#/c/6713/
http://review.gluster.org/#/c/6732/

Change-Id: I10dc12d75ec63fca313339fbc71e4f18071af552
Signed-off-by: Chetan Risbud &lt;crisbud@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6764
Reviewed-by: Prashanth Pai &lt;ppai@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Few changes have been merged to upstream swiftkerbauth repo.
This commit brings it down to gluster-swift repo.
Bringing below changes to gluster-swift repo in one go.

http://review.gluster.org/#/c/6296/
http://review.gluster.org/#/c/6370/
http://review.gluster.org/#/c/6595/
http://review.gluster.org/#/c/6713/
http://review.gluster.org/#/c/6732/

Change-Id: I10dc12d75ec63fca313339fbc71e4f18071af552
Signed-off-by: Chetan Risbud &lt;crisbud@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6764
Reviewed-by: Prashanth Pai &lt;ppai@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Initial import of the swiftkerbauth</title>
<updated>2014-01-21T18:09:44+00:00</updated>
<author>
<name>Chetan Risbud</name>
<email>crisbud@redhat.com</email>
</author>
<published>2013-12-23T10:16:22+00:00</published>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/commit/?id=4b988ce3c598c8b59bd0ce77ab7854291c66549f'/>
<id>4b988ce3c598c8b59bd0ce77ab7854291c66549f</id>
<content type='text'>
Imported code till commit f64a3354185f32928e2568d9ece4a52fa4746c05
Changed a code bit to import correct definitions.
kerbauth unit tests do run along with gluster-swift.
Install script does install swiftkerbauth.
import swiftkerbauth from http://review.gluster.org/swiftkrbauth.git

Change-Id: Ia89f2b77cc68df10dee2f41ce074f3381ac3c408
Signed-off-by: Chetan Risbud &lt;crisbud@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6597
Reviewed-by: Prashanth Pai &lt;ppai@redhat.com&gt;
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Imported code till commit f64a3354185f32928e2568d9ece4a52fa4746c05
Changed a code bit to import correct definitions.
kerbauth unit tests do run along with gluster-swift.
Install script does install swiftkerbauth.
import swiftkerbauth from http://review.gluster.org/swiftkrbauth.git

Change-Id: Ia89f2b77cc68df10dee2f41ce074f3381ac3c408
Signed-off-by: Chetan Risbud &lt;crisbud@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6597
Reviewed-by: Prashanth Pai &lt;ppai@redhat.com&gt;
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix bug where admin is able to delete reseller_admin</title>
<updated>2014-01-10T20:52:53+00:00</updated>
<author>
<name>Thiago da Silva</name>
<email>thiago@redhat.com</email>
</author>
<published>2013-12-16T17:00:38+00:00</published>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/commit/?id=205a6e4aa7e86b1e82babd3fa64d2cf002ccdaef'/>
<id>205a6e4aa7e86b1e82babd3fa64d2cf002ccdaef</id>
<content type='text'>
Changed the code to only allow the super_admin to delete a reseller_admin.
This follows the same logic of user creation, where only the super_admin
can create a reseller_admin.

Also, took the opportunity to refactor some code and implemented
get_user_detail method to remove duplicated code

Bug 1260239: https://bugs.launchpad.net/gluster-swift/+bug/1260239

Change-Id: I9e4866cd7ad08698f427846be566ab2364ad4850
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6516
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Changed the code to only allow the super_admin to delete a reseller_admin.
This follows the same logic of user creation, where only the super_admin
can create a reseller_admin.

Also, took the opportunity to refactor some code and implemented
get_user_detail method to remove duplicated code

Bug 1260239: https://bugs.launchpad.net/gluster-swift/+bug/1260239

Change-Id: I9e4866cd7ad08698f427846be566ab2364ad4850
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6516
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix users not able to change their own password/key</title>
<updated>2014-01-07T21:50:30+00:00</updated>
<author>
<name>Prashanth Pai</name>
<email>ppai@redhat.com</email>
</author>
<published>2014-01-02T06:50:20+00:00</published>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/commit/?id=94a3f539e75b069bb1f9df6f850adfe16d76b572'/>
<id>94a3f539e75b069bb1f9df6f850adfe16d76b572</id>
<content type='text'>
Users were not able to update their own password/key
with the update operation resulting in 403 (HTTPForbidden).

EXAMPLES:
Command to update password/key of regular user:
gswauth-add-user -U account1:user1 -K old_pass account1 user1 new_pass

Command to update password/key of account admin:
gswauth-add-user -U account1:admin -K old_pass -a account1 admin new_pass

Command to update password/key of reseller_admin:
gswauth-add-user -U account1:radmin -K old_pass -r account1 radmin new_pass

BUG: https://bugs.launchpad.net/gluster-swift/+bug/1262227

Change-Id: I604da5aee67099b29541eb7e51a040a041f1961b
Signed-off-by: Prashanth Pai &lt;ppai@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6650
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Users were not able to update their own password/key
with the update operation resulting in 403 (HTTPForbidden).

EXAMPLES:
Command to update password/key of regular user:
gswauth-add-user -U account1:user1 -K old_pass account1 user1 new_pass

Command to update password/key of account admin:
gswauth-add-user -U account1:admin -K old_pass -a account1 admin new_pass

Command to update password/key of reseller_admin:
gswauth-add-user -U account1:radmin -K old_pass -r account1 radmin new_pass

BUG: https://bugs.launchpad.net/gluster-swift/+bug/1262227

Change-Id: I604da5aee67099b29541eb7e51a040a041f1961b
Signed-off-by: Prashanth Pai &lt;ppai@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6650
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>gswauth: Fix 403 being returned instead of 401</title>
<updated>2013-12-18T12:12:43+00:00</updated>
<author>
<name>Prashanth Pai</name>
<email>ppai@redhat.com</email>
</author>
<published>2013-11-22T06:43:09+00:00</published>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/commit/?id=a8c84cb2da77ab294edbdc113985125f9a8acb95'/>
<id>a8c84cb2da77ab294edbdc113985125f9a8acb95</id>
<content type='text'>
- 401(Unauthorized) is to be returned when user credentials are
  wrong where as 403(Forbidden) is to be returned when user
  credentials are correct but the user doesn't have the priveleges
  to carry out the operation.

- Also error messages displayed when using swauth-* command line
  utilities have been updated.

Change-Id: I485786896ad14d3263f4325d1857cacc93adab96
Signed-off-by: Prashanth Pai &lt;ppai@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6336
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Signed-off-by: Prashanth Pai &lt;ppai@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6447
Reviewed-by: Thiago Da Silva &lt;thiago@redhat.com&gt;
Tested-by: Thiago Da Silva &lt;thiago@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
- 401(Unauthorized) is to be returned when user credentials are
  wrong where as 403(Forbidden) is to be returned when user
  credentials are correct but the user doesn't have the priveleges
  to carry out the operation.

- Also error messages displayed when using swauth-* command line
  utilities have been updated.

Change-Id: I485786896ad14d3263f4325d1857cacc93adab96
Signed-off-by: Prashanth Pai &lt;ppai@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6336
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Signed-off-by: Prashanth Pai &lt;ppai@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6447
Reviewed-by: Thiago Da Silva &lt;thiago@redhat.com&gt;
Tested-by: Thiago Da Silva &lt;thiago@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>fix issue with swauth-clean-token returning 403 errors</title>
<updated>2013-12-05T17:18:30+00:00</updated>
<author>
<name>Thiago da Silva</name>
<email>thiago@redhat.com</email>
</author>
<published>2013-12-03T23:06:21+00:00</published>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/commit/?id=2d818c06b805b051996d605ef8ef5b5415bd6293'/>
<id>2d818c06b805b051996d605ef8ef5b5415bd6293</id>
<content type='text'>
The issue was due to missing a necessary change that was made when changing
the auth account name from .auth to metadata volume.

the auth account has a group of the same name, so the .auth account
also had a .auth group, so we needed to change that too to the
metadata volume (e.g., gsmetadata)

Change-Id: Iaa3b7a1b2628f5b863807932e863593be0011a82
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6416
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The issue was due to missing a necessary change that was made when changing
the auth account name from .auth to metadata volume.

the auth account has a group of the same name, so the .auth account
also had a .auth group, so we needed to change that too to the
metadata volume (e.g., gsmetadata)

Change-Id: Iaa3b7a1b2628f5b863807932e863593be0011a82
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6416
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>fix error with swauth-clean-tokens</title>
<updated>2013-11-20T03:10:36+00:00</updated>
<author>
<name>Thiago da Silva</name>
<email>thiago@redhat.com</email>
</author>
<published>2013-11-15T22:13:53+00:00</published>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/commit/?id=f8ae648c6d557689c8c67caef6a2c6b034b7aaba'/>
<id>f8ae648c6d557689c8c67caef6a2c6b034b7aaba</id>
<content type='text'>
Error is caused by left-over code from swauth. It had hard-coded
the auth account to .auth. Changed code to use volume specified
in the config file

Change-Id: Id00eb3964b6b26a84511e4753ab80b2dea5654da
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6280
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Error is caused by left-over code from swauth. It had hard-coded
the auth account to .auth. Changed code to use volume specified
in the config file

Change-Id: Id00eb3964b6b26a84511e4753ab80b2dea5654da
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6280
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>make gluster volume name configurable</title>
<updated>2013-11-06T01:23:33+00:00</updated>
<author>
<name>Thiago da Silva</name>
<email>thiago@redhat.com</email>
</author>
<published>2013-11-04T22:08:22+00:00</published>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/commit/?id=a5c772955f5f74c2244d298641be9e2a448c9b94'/>
<id>a5c772955f5f74c2244d298641be9e2a448c9b94</id>
<content type='text'>
changed code to look at configuration file for
the metadata gluster volume name. The default is still
gsmetadata

Change-Id: I9bf74b9566ea1c9716c42f7ced0f999e02824868
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6225
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
changed code to look at configuration file for
the metadata gluster volume name. The default is still
gsmetadata

Change-Id: I9bf74b9566ea1c9716c42f7ced0f999e02824868
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6225
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>first gswauth functional tests</title>
<updated>2013-10-30T23:57:46+00:00</updated>
<author>
<name>Thiago da Silva</name>
<email>thiago@redhat.com</email>
</author>
<published>2013-10-29T21:08:03+00:00</published>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/commit/?id=9f8d2e61a79defd22fb4aa064ff5a5b936730b53'/>
<id>9f8d2e61a79defd22fb4aa064ff5a5b936730b53</id>
<content type='text'>
commiting first gswauth functional tests.
Currently there are two tests, to create account
and to create an user. Each test is self contained
in that it goes through the process of creating and deleting
accounts and users as needed.

More tests will be added shortly.

Change-Id: I26d577790aed8c79c9de11f224516423e9769962
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6188
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
commiting first gswauth functional tests.
Currently there are two tests, to create account
and to create an user. Each test is self contained
in that it goes through the process of creating and deleting
accounts and users as needed.

More tests will be added shortly.

Change-Id: I26d577790aed8c79c9de11f224516423e9769962
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6188
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>applied changes made by Luis Pabon to support gluster-swift</title>
<updated>2013-10-24T18:49:23+00:00</updated>
<author>
<name>Thiago da Silva</name>
<email>thiago@redhat.com</email>
</author>
<published>2013-10-17T19:21:59+00:00</published>
<link rel='alternate' type='text/html' href='http://dev.gluster.org/cgit/gluster-swift.git/commit/?id=b48149a4af9bc2c5de14a4f479bc7b11938bcf94'/>
<id>b48149a4af9bc2c5de14a4f479bc7b11938bcf94</id>
<content type='text'>
The current changes support only one account per volume
The next changes will be made to add support for multiple accounts
Also modified unit tests that were failing after code changes.
Original changes were made here: https://github.com/lpabon/swauth/tree/gluster-swift

Change-Id: I5577d7dc042fc22de8625c8bdf30329cc3338cb9
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6107
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The current changes support only one account per volume
The next changes will be made to add support for multiple accounts
Also modified unit tests that were failing after code changes.
Original changes were made here: https://github.com/lpabon/swauth/tree/gluster-swift

Change-Id: I5577d7dc042fc22de8625c8bdf30329cc3338cb9
Signed-off-by: Thiago da Silva &lt;thiago@redhat.com&gt;
Reviewed-on: http://review.gluster.org/6107
Reviewed-by: Luis Pabon &lt;lpabon@redhat.com&gt;
Tested-by: Luis Pabon &lt;lpabon@redhat.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
