From dca811d6cbce42b0caba025da12459c925cf9065 Mon Sep 17 00:00:00 2001
From: Marcin Kuzminski <marcin@rhodecode.com>
Date: Tue, 24 May 2016 17:24:38 +0200
Subject: [PATCH 01/11] version: bumped to 4.0.1

---
 .bumpversion.cfg  | 3 ++-
 rhodecode/VERSION | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/.bumpversion.cfg b/.bumpversion.cfg
index fca45ff4..a453155a 100644
--- a/.bumpversion.cfg
+++ b/.bumpversion.cfg
@@ -1,5 +1,6 @@
 [bumpversion]
-current_version = 4.0.0
+current_version = 4.0.1
 message = release: Bump version {current_version} to {new_version}
 
 [bumpversion:file:rhodecode/VERSION]
+
diff --git a/rhodecode/VERSION b/rhodecode/VERSION
index 0c89fc92..cc868b62 100644
--- a/rhodecode/VERSION
+++ b/rhodecode/VERSION
@@ -1 +1 @@
-4.0.0
\ No newline at end of file
+4.0.1
\ No newline at end of file

From a31dc2a19e0cfefe7a5dc98ff7e5ad448b8cd601 Mon Sep 17 00:00:00 2001
From: Daniel Dourvaris <daniel@rhodecode.com>
Date: Tue, 24 May 2016 19:21:56 +0300
Subject: [PATCH 02/11] pullrequests: select a ref if one exists matching the
 commit id

---
 rhodecode/model/pull_request.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/rhodecode/model/pull_request.py b/rhodecode/model/pull_request.py
index 535d6c43..e25a79e5 100644
--- a/rhodecode/model/pull_request.py
+++ b/rhodecode/model/pull_request.py
@@ -1054,8 +1054,10 @@ class PullRequestModel(BaseModel):
                 ref_key = '%s:%s:%s' % (group_key, ref_name, ref_id)
                 group_refs.append((ref_key, ref_name))
 
-                if not selected and match in (ref_id, ref_name):
-                    selected = ref_key
+                if not selected:
+                    if set([commit_id, match]) & set([ref_id, ref_name]):
+                        selected = ref_key
+
             if group_refs:
                 groups.append((group_refs, group_name))
 

From d7511e433774cfdf679136379025e7b6dc14d99a Mon Sep 17 00:00:00 2001
From: lisaq <lisa@rhodecode.com>
Date: Tue, 24 May 2016 17:28:17 +0200
Subject: [PATCH 03/11] html: fixing forks table #3959

---
 rhodecode/templates/forks/forks_data.html | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/rhodecode/templates/forks/forks_data.html b/rhodecode/templates/forks/forks_data.html
index 66aad2a1..d505cc2c 100644
--- a/rhodecode/templates/forks/forks_data.html
+++ b/rhodecode/templates/forks/forks_data.html
@@ -4,6 +4,7 @@
 % if c.forks_pager:
   <table class="rctable fork_summary">
     <tr>
+      <th>${_('Owner')}</th>
       <th>${_('Fork')}</th>
       <th>${_('Description')}</th>
       <th>${_('Forked')}</th>
@@ -13,7 +14,8 @@
         <tr>
             <td class="td-user fork_user">
               ${base.gravatar_with_user(f.user.email, 16)}
-               &frasl;
+            </td>
+            <td class="td-componentname">
               ${h.link_to(f.repo_name,h.url('summary_home',repo_name=f.repo_name))}
             </td>
             <td class="td-description">

From 83f9357645f5ab63473971a1b2617d75b1b1dec4 Mon Sep 17 00:00:00 2001
From: lisaq <lisa@rhodecode.com>
Date: Tue, 24 May 2016 18:17:14 +0200
Subject: [PATCH 04/11] styling: gravatar improvements for special cases

---
 rhodecode/public/css/code-block.less                |  5 +++++
 rhodecode/public/css/comments.less                  |  5 +++++
 rhodecode/public/css/main.less                      | 13 ++++++++++++-
 .../templates/pullrequests/pullrequest_show.html    |  6 +++---
 4 files changed, 25 insertions(+), 4 deletions(-)

diff --git a/rhodecode/public/css/code-block.less b/rhodecode/public/css/code-block.less
index d26f452a..c2d8d892 100644
--- a/rhodecode/public/css/code-block.less
+++ b/rhodecode/public/css/code-block.less
@@ -439,6 +439,11 @@ div.codeblock {
         padding: @padding;
         border-bottom: @border-thickness solid @grey5;
 
+        .rc-user {
+            min-width: 0;
+            margin-right: .5em;
+        }
+
         .stats {
             clear: both;
             margin: 0 0 @padding 0;
diff --git a/rhodecode/public/css/comments.less b/rhodecode/public/css/comments.less
index b0241961..07d60ed7 100644
--- a/rhodecode/public/css/comments.less
+++ b/rhodecode/public/css/comments.less
@@ -52,6 +52,11 @@ tr.inline-comments div {
     padding: 4px 0 0 0;
     line-height: 1em;
 
+    .rc-user {
+        min-width: 0;
+        margin: -2px .5em 0 0;
+    }
+
     .meta {
         position: relative;
         width: 100%;
diff --git a/rhodecode/public/css/main.less b/rhodecode/public/css/main.less
index 317390e2..df6762f9 100644
--- a/rhodecode/public/css/main.less
+++ b/rhodecode/public/css/main.less
@@ -671,6 +671,7 @@ label {
 }
 
 .rc-user { // gravatar + user wrapper
+    float: left;
     position: relative;
     min-width: 100px;
     max-width: 200px;
@@ -1223,6 +1224,16 @@ table.issuetracker {
     max-width: 83%;
     padding-right: 20px;
     vertical-align: middle;
+    line-height: 1;
+
+    .rc-user {
+        min-width: 0;
+        margin: -2px 1em 0 0;
+    }
+
+    .reviewer {
+        float: left;
+    }
 }
 
 .reviewer_member_remove {
@@ -1253,7 +1264,7 @@ table.issuetracker {
 }
 .pr-details-content {
     margin-top: @textmargin;
-    margin-bottom: @textmargin/2;
+    margin-bottom: @textmargin;
 }
 .pr-description {
     white-space:pre-wrap;
diff --git a/rhodecode/templates/pullrequests/pullrequest_show.html b/rhodecode/templates/pullrequests/pullrequest_show.html
index 21d471e0..dcddb8e6 100644
--- a/rhodecode/templates/pullrequests/pullrequest_show.html
+++ b/rhodecode/templates/pullrequests/pullrequest_show.html
@@ -195,9 +195,9 @@
                     <div class="reviewer_status tooltip" title="${h.tooltip(h.commit_status_lbl(status[0][1].status if status else 'not_reviewed'))}">
                       <div class="${'flag_status %s' % (status[0][1].status if status else 'not_reviewed')} pull-left reviewer_member_status"></div>
                     </div>
-                  <span id="reviewer_${member.user_id}_name" class="reviewer_name">
-                    ${self.gravatar_with_user(member.email, 16)}
-                    (${_('owner') if c.pull_request.user_id == member.user_id else _('reviewer')})</span>
+                  <div id="reviewer_${member.user_id}_name" class="reviewer_name">
+                    ${self.gravatar_with_user(member.email, 16)} <div class="reviewer">(${_('owner') if c.pull_request.user_id == member.user_id else _('reviewer')})</div>
+                  </div>
                   <input id="reviewer_${member.user_id}_input" type="hidden" value="${member.user_id}" name="review_members" />
                   %if c.allowed_to_update:
                   <div class="reviewer_member_remove action_button" onclick="removeReviewMember(${member.user_id}, true)" style="visibility: hidden;">

From 8b9d6731436f4b563a26fb1c9636b8d6f4240931 Mon Sep 17 00:00:00 2001
From: Johannes Bornhold <johannes@rhodecode.com>
Date: Tue, 24 May 2016 23:47:50 +0200
Subject: [PATCH 05/11] dependencies: Add gunicorn

Adding gunicorn into the default environment so that we have it available when
dunning nix-shell as well.
---
 pkgs/python-packages.nix | 10 +++++-----
 setup.py                 |  2 ++
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/pkgs/python-packages.nix b/pkgs/python-packages.nix
index 422fec95..9848530a 100644
--- a/pkgs/python-packages.nix
+++ b/pkgs/python-packages.nix
@@ -360,7 +360,7 @@
     };
   };
   certifi = super.buildPythonPackage {
-    name = "certifi-2016.02.28";
+    name = "certifi-2016.2.28";
     buildInputs = with self; [];
     doCheck = false;
     propagatedBuildInputs = with self; [];
@@ -1050,10 +1050,10 @@
     };
   };
   rhodecode-enterprise-ce = super.buildPythonPackage {
-    name = "rhodecode-enterprise-ce-4.0.0";
-    buildInputs = with self; [WebTest configobj cssselect flake8 lxml mock pytest pytest-runner pytest-cov];
+    name = "rhodecode-enterprise-ce-4.0.1";
+    buildInputs = with self; [WebTest configobj cssselect flake8 lxml mock pytest pytest-cov pytest-runner];
     doCheck = true;
-    propagatedBuildInputs = with self; [Babel Beaker FormEncode Mako Markdown MarkupSafe MySQL-python Paste PasteDeploy PasteScript Pygments Pylons Pyro4 Routes SQLAlchemy Tempita URLObject WebError WebHelpers WebHelpers2 WebOb WebTest Whoosh alembic amqplib anyjson appenlight-client authomatic backport-ipaddress celery colander decorator docutils infrae.cache ipython iso8601 kombu msgpack-python packaging psycopg2 pycrypto pycurl pyparsing pyramid pyramid-debugtoolbar pyramid-mako pyramid-beaker pysqlite python-dateutil python-ldap python-memcached python-pam recaptcha-client repoze.lru requests simplejson waitress zope.cachedescriptors psutil py-bcrypt];
+    propagatedBuildInputs = with self; [Babel Beaker FormEncode Mako Markdown MarkupSafe MySQL-python Paste PasteDeploy PasteScript Pygments Pylons Pyro4 Routes SQLAlchemy Tempita URLObject WebError WebHelpers WebHelpers2 WebOb WebTest Whoosh alembic amqplib anyjson appenlight-client authomatic backport-ipaddress celery colander decorator docutils gunicorn infrae.cache ipython iso8601 kombu msgpack-python packaging psycopg2 pycrypto pycurl pyparsing pyramid pyramid-debugtoolbar pyramid-mako pyramid-beaker pysqlite python-dateutil python-ldap python-memcached python-pam recaptcha-client repoze.lru requests simplejson waitress zope.cachedescriptors psutil py-bcrypt];
     src = ./.;
   };
   rhodecode-tools = super.buildPythonPackage {
@@ -1269,5 +1269,5 @@
 
 ### Test requirements
 
-
+  
 }
diff --git a/setup.py b/setup.py
index a0c03b6d..ccc02753 100644
--- a/setup.py
+++ b/setup.py
@@ -73,6 +73,7 @@ requirements = [
     'colander',
     'decorator',
     'docutils',
+    'gunicorn',
     'infrae.cache',
     'ipython',
     'iso8601',
@@ -114,6 +115,7 @@ test_requirements = [
     'lxml',
     'mock',
     'pytest',
+    'pytest-cov',
     'pytest-runner',
 ]
 

From b0695a1fdcdd50b5e1cf9852c6de304703dc05cd Mon Sep 17 00:00:00 2001
From: Marcin Kuzminski <marcin@rhodecode.com>
Date: Tue, 24 May 2016 23:11:47 +0200
Subject: [PATCH 06/11] logging: fixed log formatting

---
 rhodecode/lib/middleware/simplevcs.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/rhodecode/lib/middleware/simplevcs.py b/rhodecode/lib/middleware/simplevcs.py
index 1534a118..5103fdfe 100644
--- a/rhodecode/lib/middleware/simplevcs.py
+++ b/rhodecode/lib/middleware/simplevcs.py
@@ -133,7 +133,8 @@ class SimpleVCS(object):
     def is_valid_and_existing_repo(self, repo_name, base_path, scm_type):
         db_repo = Repository.get_by_repo_name(repo_name)
         if not db_repo:
-            log.debug('Repository `%s` not found inside the database.')
+            log.debug('Repository `%s` not found inside the database.',
+                      repo_name)
             return False
 
         if db_repo.repo_type != scm_type:

From 825f205dfb94582691a85484bafd91fd5837a4e7 Mon Sep 17 00:00:00 2001
From: Marcin Kuzminski <marcin@rhodecode.com>
Date: Tue, 24 May 2016 23:46:55 +0200
Subject: [PATCH 07/11] configs: show worker_connection option for gunicorn

---
 configs/development.ini | 2 ++
 configs/production.ini  | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/configs/development.ini b/configs/development.ini
index c9f2c3a9..146a4007 100644
--- a/configs/development.ini
+++ b/configs/development.ini
@@ -65,6 +65,8 @@ asyncore_use_poll = true
 ## type of worker class, one of sync, gevent
 ## recommended for bigger setup is using of of other than sync one
 #worker_class = sync
+## The maximum number of simultaneous clients. Valid only for Gevent
+#worker_connections = 10
 ## max number of requests that worker will handle before being gracefully
 ## restarted, could prevent memory leaks
 #max_requests = 1000
diff --git a/configs/production.ini b/configs/production.ini
index 856e442f..1802bb83 100644
--- a/configs/production.ini
+++ b/configs/production.ini
@@ -65,6 +65,8 @@ asyncore_use_poll = true
 ## type of worker class, one of sync, gevent
 ## recommended for bigger setup is using of of other than sync one
 #worker_class = sync
+## The maximum number of simultaneous clients. Valid only for Gevent
+#worker_connections = 10
 ## max number of requests that worker will handle before being gracefully
 ## restarted, could prevent memory leaks
 #max_requests = 1000

From 19ef5c2721dbd7586acce10e8a9180aee8078cb7 Mon Sep 17 00:00:00 2001
From: Marcin Kuzminski <marcin@rhodecode.com>
Date: Tue, 24 May 2016 23:47:32 +0200
Subject: [PATCH 08/11] authn: don't use formatted_json to log statements. It
 totally screws up logging formatting, and makes big entries in logs with
 newlines.

---
 rhodecode/authentication/plugins/auth_jasig_cas.py | 9 +++------
 rhodecode/authentication/plugins/auth_ldap.py      | 3 +--
 rhodecode/authentication/plugins/auth_pam.py       | 3 +--
 3 files changed, 5 insertions(+), 10 deletions(-)

diff --git a/rhodecode/authentication/plugins/auth_jasig_cas.py b/rhodecode/authentication/plugins/auth_jasig_cas.py
index 36c24c1f..c537d20f 100644
--- a/rhodecode/authentication/plugins/auth_jasig_cas.py
+++ b/rhodecode/authentication/plugins/auth_jasig_cas.py
@@ -36,7 +36,6 @@ from sqlalchemy.ext.hybrid import hybrid_property
 from rhodecode.authentication.base import RhodeCodeExternalAuthPlugin
 from rhodecode.authentication.schema import AuthnPluginSettingsSchemaBase
 from rhodecode.authentication.routes import AuthnPluginResourceBase
-from rhodecode.lib.ext_json import formatted_json
 from rhodecode.lib.utils2 import safe_unicode
 from rhodecode.model.db import User
 
@@ -119,17 +118,15 @@ class RhodeCodeAuthPlugin(RhodeCodeExternalAuthPlugin):
             log.debug('Empty username or password skipping...')
             return None
 
-        log.debug("Jasig CAS settings: \n%s" % (formatted_json(settings)))
+        log.debug("Jasig CAS settings: %s", settings)
         params = urllib.urlencode({'username': username, 'password': password})
         headers = {"Content-type": "application/x-www-form-urlencoded",
                    "Accept": "text/plain",
                    "User-Agent": "RhodeCode-auth-%s" % rhodecode.__version__}
         url = settings["service_url"]
 
-        log.debug("Sent Jasig CAS: \n%s"
-                  % (formatted_json({"url": url,
-                                     "body": params,
-                                     "headers": headers})))
+        log.debug("Sent Jasig CAS: \n%s",
+                  {"url": url, "body": params, "headers": headers})
         request = urllib2.Request(url, params, headers)
         try:
             response = urllib2.urlopen(request)
diff --git a/rhodecode/authentication/plugins/auth_ldap.py b/rhodecode/authentication/plugins/auth_ldap.py
index 3303ca7c..8e7a1f02 100644
--- a/rhodecode/authentication/plugins/auth_ldap.py
+++ b/rhodecode/authentication/plugins/auth_ldap.py
@@ -36,7 +36,6 @@ from rhodecode.authentication.routes import AuthnPluginResourceBase
 from rhodecode.lib.exceptions import (
     LdapConnectionError, LdapUsernameError, LdapPasswordError, LdapImportError
 )
-from rhodecode.lib.ext_json import formatted_json
 from rhodecode.lib.utils2 import safe_unicode, safe_str
 from rhodecode.model.db import User
 from rhodecode.model.validators import Missing
@@ -435,7 +434,7 @@ class RhodeCodeAuthPlugin(RhodeCodeExternalAuthPlugin):
                 'extern_name': user_dn,
                 'extern_type': extern_type,
             }
-            log.debug('ldap user: \n%s', formatted_json(user_attrs))
+            log.debug('ldap user: %s', user_attrs)
             log.info('user %s authenticated correctly', user_attrs['username'])
 
             return user_attrs
diff --git a/rhodecode/authentication/plugins/auth_pam.py b/rhodecode/authentication/plugins/auth_pam.py
index 3b461548..21995959 100644
--- a/rhodecode/authentication/plugins/auth_pam.py
+++ b/rhodecode/authentication/plugins/auth_pam.py
@@ -35,7 +35,6 @@ from sqlalchemy.ext.hybrid import hybrid_property
 from rhodecode.authentication.base import RhodeCodeExternalAuthPlugin
 from rhodecode.authentication.schema import AuthnPluginSettingsSchemaBase
 from rhodecode.authentication.routes import AuthnPluginResourceBase
-from rhodecode.lib.ext_json import formatted_json
 
 log = logging.getLogger(__name__)
 
@@ -151,6 +150,6 @@ class RhodeCodeAuthPlugin(RhodeCodeExternalAuthPlugin):
             log.warning("Cannot extract additional info for PAM user")
             pass
 
-        log.debug("pamuser: \n%s" % formatted_json(user_attrs))
+        log.debug("pamuser: %s", user_attrs)
         log.info('user %s authenticated correctly' % user_attrs['username'])
         return user_attrs

From 2deb73cf7027fc065f37660707a1890c9b2a57b2 Mon Sep 17 00:00:00 2001
From: Marcin Kuzminski <marcin@rhodecode.com>
Date: Wed, 25 May 2016 10:00:27 +0200
Subject: [PATCH 09/11] makefile: added commands to quickly build docs.

---
 Makefile | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/Makefile b/Makefile
index fdcb2c13..7c80223c 100644
--- a/Makefile
+++ b/Makefile
@@ -5,10 +5,17 @@ NODE_PATH=./node_modules
 FLAKE8=flake8 setup.py pytest_pylons/ rhodecode/ --select=E124 --ignore=E711,E712,E510,E121,E122,E126,E127,E128,E501,F401 --max-line-length=100 --exclude=*rhodecode/lib/dbmigrate/*,*rhodecode/tests/*,*rhodecode/lib/vcs/utils/*
 CI_PREFIX=enterprise
 
-.PHONY: help clean test test-clean test-lint test-only
+.PHONY: docs docs-clean ci-docs clean test test-clean test-lint test-only
+
+
+docs:
+	(cd docs; nix-build default.nix -o result; make clean html)
+
+docs-clean:
+	(cd docs; make clean)
+
+ci-docs: docs;
 
-help:
-	@echo "TODO: describe Makefile"
 
 clean: test-clean
 	find . -type f \( -iname '*.c' -o -iname '*.pyc' -o -iname '*.so' \) -exec rm '{}' ';'
@@ -37,3 +44,4 @@ web-test:
 docs-bootstrap:
 	(cd docs; nix-build default.nix -o result)
 	@echo "Please go to docs folder and run make html"
+

From 9e895813c6347de7e487edf7d4164d3f715607ec Mon Sep 17 00:00:00 2001
From: Marcin Kuzminski <marcin@rhodecode.com>
Date: Wed, 25 May 2016 12:37:56 +0200
Subject: [PATCH 10/11] configs: updated default config.

- default for Beaker suddenly is memory, and we shoould make file a default.
---
 configs/production.ini | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/configs/production.ini b/configs/production.ini
index 1802bb83..29296d78 100644
--- a/configs/production.ini
+++ b/configs/production.ini
@@ -290,12 +290,12 @@ beaker.cache.repo_cache_long.expire = 2592000
 ####################################
 
 ## .session.type is type of storage options for the session, current allowed
-## types are file(default), ext:memcached, ext:database, and memory.
-#beaker.session.type = file
+## types are file, ext:memcached, ext:database, and memory(default).
+beaker.session.type = file
+beaker.session.data_dir = %(here)s/data/sessions/data
 
 ## db based session, fast, and allows easy management over logged in users ##
 #beaker.session.type = ext:database
-#beaker.session.lock_dir = %(here)s/data/cache/session_db_lock
 #beaker.session.table_name = db_session
 #beaker.session.sa.url = postgresql://postgres:secret@localhost/rhodecode
 #beaker.session.sa.url = mysql://root:secret@127.0.0.1/rhodecode
@@ -304,6 +304,7 @@ beaker.cache.repo_cache_long.expire = 2592000
 
 beaker.session.key = rhodecode
 beaker.session.secret = production-rc-uytcxaz
+#beaker.session.lock_dir = %(here)s/data/sessions/lock
 
 ## Secure encrypted cookie. Requires AES and AES python libraries
 ## you must disable beaker.session.secret to use this
@@ -437,6 +438,13 @@ vcs.server.enable = true
 vcs.server = localhost:9900
 # Available protocols: pyro4, http
 vcs.server.protocol = pyro4
+
+# available impl:
+# vcsserver.scm_app (EE only, for testing),
+# rhodecode.lib.middleware.utils.scm_app_http
+# pyro4
+#vcs.scm_app_implementation = rhodecode.lib.middleware.utils.scm_app_http
+
 vcs.server.log_level = info
 vcs.start_server = false
 vcs.backends = hg, git, svn

From 12139a5164389b65fd3c1bf5d31a133d4bb35173 Mon Sep 17 00:00:00 2001
From: Marcin Kuzminski <marcin@rhodecode.com>
Date: Wed, 25 May 2016 12:46:45 +0200
Subject: [PATCH 11/11] docs: added 4.0.1 release notes

---
 docs/release-notes/release-notes-4.0.1.rst | 17 +++++++++++++++++
 docs/release-notes/release-notes.rst       |  1 +
 2 files changed, 18 insertions(+)
 create mode 100644 docs/release-notes/release-notes-4.0.1.rst

diff --git a/docs/release-notes/release-notes-4.0.1.rst b/docs/release-notes/release-notes-4.0.1.rst
new file mode 100644
index 00000000..05fc1c50
--- /dev/null
+++ b/docs/release-notes/release-notes-4.0.1.rst
@@ -0,0 +1,17 @@
+|RCE| 4.0.1 |RNS|
+-----------------
+
+Release Date
+^^^^^^^^^^^^
+
+- 2016-05-25
+
+Fixes
+^^^^^
+
+- fixed default session to be file based instead of memory which causes
+  problems in multi-worker setup
+- ui: fixing forks table #3959
+- ui: fixed gravatars misalignment issues
+- logging: fixed excesive formatting on auth logging
+- pull requests: better ref selection  when opening PRs from changelog
diff --git a/docs/release-notes/release-notes.rst b/docs/release-notes/release-notes.rst
index 2b51ebc6..6c193661 100644
--- a/docs/release-notes/release-notes.rst
+++ b/docs/release-notes/release-notes.rst
@@ -6,6 +6,7 @@ Release Notes
 |RCE| 4.x Versions
 ------------------
 
+   release-notes-4.0.1.rst
    release-notes-4.0.0.rst
 
 |RCE| 3.x Versions