You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@couchdb.apache.org by mikewallace1979 <gi...@git.apache.org> on 2016/02/09 16:10:39 UTC

[GitHub] couchdb-fabric pull request: Return forbidden error if encountered...

GitHub user mikewallace1979 opened a pull request:

    https://github.com/apache/couchdb-fabric/pull/38

    Return forbidden error if encountered on any shard

    This commit fixes an issue which caused HTTP 500 errors to be
    returned when an authorized user attempted to access a database
    that they did not have permission to access, when cassim is
    disabled.
    
    Instead of returning an internal server error once all shards
    have failed to open we add a receive clause so that we throw
    a forbidden error if one is encountered on any shard. This is the
    same approach we already take for unauthorized errors.
    
    Closes COUCHDB-2948

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/mikewallace1979/couchdb-fabric 2948-return-forbidden-error-if-encountered-on-any-shard

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/couchdb-fabric/pull/38.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #38
    
----
commit cb6e82a6f9c1827f3ef9f880d927183fad61333f
Author: Mike Wallace <mi...@apache.org>
Date:   2016-02-09T14:31:55Z

    Return forbidden error if encountered on any shard
    
    This commit fixes an issue which caused HTTP 500 errors to be
    returned when an authorized user attempted to access a database
    that they did not have permission to access, when cassim is
    disabled.
    
    Instead of returning an internal server error once all shards
    have failed to open we add a receive clause so that we throw
    a forbidden error if one is encountered on any shard. This is the
    same approach we already take for unauthorized errors.
    
    Closes COUCHDB-2948

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] couchdb-fabric pull request: Return forbidden error if encountered...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/couchdb-fabric/pull/38


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] couchdb-fabric pull request: Return forbidden error if encountered...

Posted by kxepal <gi...@git.apache.org>.
Github user kxepal commented on the pull request:

    https://github.com/apache/couchdb-fabric/pull/38#issuecomment-181912838
  
    +1


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---