[ARVADOS] updated: 2553fde59a3cf872be891a4f689c241055080c35

git at public.curoverse.com git at public.curoverse.com
Mon Aug 10 11:40:58 EDT 2015


Summary of changes:
 sdk/python/arvados/arvfile.py           |   4 +-
 sdk/python/arvados/commands/arv_copy.py |  36 ++++-----
 sdk/python/arvados/keep.py              |   5 +-
 sdk/python/arvados/retry.py             |   3 +-
 sdk/python/tests/test_retry.py          |   6 +-
 sdk/python/tests/test_websockets.py     | 136 +++++++++++++++-----------------
 services/api/test/fixtures/humans.yml   |   1 +
 services/arv-git-httpd/auth_handler.go  |   2 +
 8 files changed, 95 insertions(+), 98 deletions(-)
 create mode 100644 services/api/test/fixtures/humans.yml

       via  2553fde59a3cf872be891a4f689c241055080c35 (commit)
       via  63b82f437c0f237c76f460ef71be0bb3ed42f7f4 (commit)
       via  2a96c097e5a176018d078a5d6985403072e8672e (commit)
       via  970907f28866a09a9fe95da48dffa6cd34ab4dca (commit)
       via  9209660a3bd951c3945bce0da2fa9195cb002e44 (commit)
       via  0988acb472849dc08d576ee40493e70bde2132ca (commit)
       via  f46b05c041684ced4cf438ae6cade577156f81a5 (commit)
       via  fc851b249ea25a40a1fb392906705142113ac5b9 (commit)
       via  d2d7138c84a55ef87937cdaefd9c58a66916d76f (commit)
      from  3959d7afff8bb3c3b8da9eb7d178919275180f2a (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 2553fde59a3cf872be891a4f689c241055080c35
Author: radhika <radhika at curoverse.com>
Date:   Mon Aug 10 11:40:24 2015 -0400

    6827: add a comment that only the first 10 characters of a valid token are logged.

diff --git a/services/arv-git-httpd/auth_handler.go b/services/arv-git-httpd/auth_handler.go
index 7463548..b284f19 100644
--- a/services/arv-git-httpd/auth_handler.go
+++ b/services/arv-git-httpd/auth_handler.go
@@ -53,6 +53,8 @@ func (h *authHandler) ServeHTTP(wOrig http.ResponseWriter, r *http.Request) {
 			w.Write([]byte(statusText))
 		}
 
+    // If the given password is a valid token, log the first 10 characters of the token.
+    // Otherwise: log the string <invalid> if a password is given, else an empty string.
 		passwordToLog := ""
 		if statusCode == 401 || strings.Contains(statusText, "Unauthorized") {
 			if len(password) > 0 {

commit 63b82f437c0f237c76f460ef71be0bb3ed42f7f4
Merge: 3959d7a 2a96c09
Author: radhika <radhika at curoverse.com>
Date:   Mon Aug 10 11:33:32 2015 -0400

    Merge branch 'master' into 6827-no-passwords-in-logs


-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list