Uploaded image for project: 'CouchDB'
  1. CouchDB
  2. COUCHDB-2731

"authentication_db" for clustered interface is not considered as sysdb

    Details

    • Type: Bug
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      In case config:get("chttpd_auth", "authentication_db", "_users") returns a non default db name. The configured database is not considered a user's db. Which means callbacks such as `couch_users_db:before_doc_update` are not used.

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user iilyak commented on the pull request:

          https://github.com/apache/couchdb-couch/pull/62#issuecomment-115856279

          The explanation is in a bug description https://issues.apache.org/jira/browse/COUCHDB-2731

          Show
          githubbot ASF GitHub Bot added a comment - Github user iilyak commented on the pull request: https://github.com/apache/couchdb-couch/pull/62#issuecomment-115856279 The explanation is in a bug description https://issues.apache.org/jira/browse/COUCHDB-2731
          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user iilyak opened a pull request:

          https://github.com/apache/couchdb-couch/pull/198

          Make sure authentication_db for clustered is considered as users db

          Fixes the case when the following are configured to something other than _users

          • chttpd_auth.authentication_db
          • couch_httpd_auth.authentication_db

          For chttpd_auth only clustered db should be considered as users_db.
          For couch_http_db only node-local db should be considered.
          So we take wheither it is a clustered db into account in `is_usersdb/2`.

          COUCHDB-2731

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

          $ git pull https://github.com/cloudant/couchdb-couch 74046-authentication_db-should-be-sysdb

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

          https://github.com/apache/couchdb-couch/pull/198.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 #198


          commit ef2ff60d3430ead8101102608c1684ceba287046
          Author: ILYA Khlopotov <iilyak@ca.ibm.com>
          Date: 2016-09-13T18:42:47Z

          Make sure authentication_db for clustered is considered as users db

          Fixes the case when the following are configured to something other than _users

          • chttpd_auth.authentication_db
          • couch_httpd_auth.authentication_db

          For chttpd_auth only clustered db should be considered as users_db.
          For couch_http_db only node-local db should be considered.
          So we take wheither it is a clustered db into account in `is_usersdb/2`.

          COUCHDB-2731


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user iilyak opened a pull request: https://github.com/apache/couchdb-couch/pull/198 Make sure authentication_db for clustered is considered as users db Fixes the case when the following are configured to something other than _users chttpd_auth.authentication_db couch_httpd_auth.authentication_db For chttpd_auth only clustered db should be considered as users_db. For couch_http_db only node-local db should be considered. So we take wheither it is a clustered db into account in `is_usersdb/2`. COUCHDB-2731 You can merge this pull request into a Git repository by running: $ git pull https://github.com/cloudant/couchdb-couch 74046-authentication_db-should-be-sysdb Alternatively you can review and apply these changes as the patch at: https://github.com/apache/couchdb-couch/pull/198.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 #198 commit ef2ff60d3430ead8101102608c1684ceba287046 Author: ILYA Khlopotov <iilyak@ca.ibm.com> Date: 2016-09-13T18:42:47Z Make sure authentication_db for clustered is considered as users db Fixes the case when the following are configured to something other than _users chttpd_auth.authentication_db couch_httpd_auth.authentication_db For chttpd_auth only clustered db should be considered as users_db. For couch_http_db only node-local db should be considered. So we take wheither it is a clustered db into account in `is_usersdb/2`. COUCHDB-2731
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit f72e607c6a8d3b2a345bfeacb7a5c9d6f3216c2b in couchdb-couch's branch refs/heads/master from ILYA
          [ https://git-wip-us.apache.org/repos/asf?p=couchdb-couch.git;h=f72e607 ]

          Make sure clustered authentication_db is sys_db

          Fixes the case when the following are configured to something other than _users

          • chttpd_auth.authentication_db
          • couch_httpd_auth.authentication_db

          Introduce "couchdb.users_db_suffix" config key

          Remove distinction between clustered and local users databases
          configuration. We remove following configuration options and replace
          them with "couchdb.users_db_suffix":

          • chttpd_auth.authentication_db
          • couch_httpd_auth.authentication_db

          COUCHDB-2731

          Show
          jira-bot ASF subversion and git services added a comment - Commit f72e607c6a8d3b2a345bfeacb7a5c9d6f3216c2b in couchdb-couch's branch refs/heads/master from ILYA [ https://git-wip-us.apache.org/repos/asf?p=couchdb-couch.git;h=f72e607 ] Make sure clustered authentication_db is sys_db Fixes the case when the following are configured to something other than _users chttpd_auth.authentication_db couch_httpd_auth.authentication_db Introduce "couchdb.users_db_suffix" config key Remove distinction between clustered and local users databases configuration. We remove following configuration options and replace them with "couchdb.users_db_suffix": chttpd_auth.authentication_db couch_httpd_auth.authentication_db COUCHDB-2731
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/couchdb-couch/pull/198

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/couchdb-couch/pull/198

            People

            • Assignee:
              iilyak ILYA
              Reporter:
              iilyak ILYA
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:

                Development