Commit Graph

493 Commits

Author SHA1 Message Date
Girish Ramakrishnan
f4958d936c return groupIds in get user route 2016-06-03 00:00:11 -07:00
Girish Ramakrishnan
80ca69a128 user.update does not need the user object 2016-06-02 23:53:06 -07:00
Girish Ramakrishnan
097d23c412 move logic to model code 2016-06-02 23:29:43 -07:00
Girish Ramakrishnan
13a1213b0d make group listing API return member userIds 2016-06-02 21:07:33 -07:00
Girish Ramakrishnan
8d65f93fa4 return error.message 2016-06-02 14:40:29 -07:00
Girish Ramakrishnan
462440bb30 do not check for password in profile route
This is already checked by the verifyPassword middleware based on
the token type.

When using dev tokens, this check barfs for lack of password field
even when none is required.
2016-06-02 14:26:01 -07:00
Girish Ramakrishnan
65261dc4d5 add time_zone setter route 2016-06-02 13:54:07 -07:00
Girish Ramakrishnan
28b3550214 use error.message 2016-06-02 13:00:23 -07:00
Johannes Zellner
7326ea27ca Only set username and displayName after successful update 2016-06-02 21:12:02 +02:00
Girish Ramakrishnan
1fe00f7f80 do not use verbs in resource url 2016-06-02 12:01:48 -07:00
Girish Ramakrishnan
e9e9d6000d remove token check for user.update to work with dev tokens 2016-06-02 11:29:59 -07:00
Johannes Zellner
bdd14022d6 Report user conflict message all the way through the rest routes 2016-06-02 15:41:07 +02:00
Johannes Zellner
8eeee712aa Remove unused require 2016-06-02 14:14:16 +02:00
Johannes Zellner
0f62faa198 All our tokens are now representing an user with a profile 2016-06-02 14:13:57 +02:00
Girish Ramakrishnan
60ce6b69ee profile updates must be POST 2016-06-02 00:31:41 -07:00
Girish Ramakrishnan
4fcc7fe99f updateUser is POST 2016-06-02 00:27:06 -07:00
Girish Ramakrishnan
82cd215ffa merge bad fields and pass error.message correctly in REST responses 2016-06-02 00:12:21 -07:00
Girish Ramakrishnan
9cc6cb56f7 fix error message 2016-06-01 19:38:42 -07:00
Girish Ramakrishnan
824767adbb enabled -> isEnabled 2016-06-01 18:21:02 -07:00
Girish Ramakrishnan
b13de298bf Add some REST api tests 2016-06-01 16:33:18 -07:00
Girish Ramakrishnan
f763759008 return empty groupIds 2016-05-31 11:49:59 -07:00
Girish Ramakrishnan
ce927bfa22 alias
also remove id since it's not useful for mailbox case (not like
mailbox can be renamed and we need a fixed it)
2016-05-27 22:20:08 -07:00
Girish Ramakrishnan
6993a9c7e7 add more mailbox route test 2016-05-27 18:23:14 -07:00
Girish Ramakrishnan
84d04cce16 initial mailboxes route 2016-05-27 18:17:57 -07:00
Girish Ramakrishnan
4afdf50736 finally finally finally the tests are working 2016-05-24 20:04:37 -07:00
Girish Ramakrishnan
b4ba17c599 use the constant 2016-05-24 13:23:41 -07:00
Girish Ramakrishnan
57a823a698 make tests work 2016-05-24 00:44:01 -07:00
Girish Ramakrishnan
ec0ee07b17 test: email works now 2016-05-23 23:17:38 -07:00
Girish Ramakrishnan
3d7545133e wait 30 secs in the beginning 2016-05-23 23:16:02 -07:00
Girish Ramakrishnan
bcc752469a remove containers after the test 2016-05-23 22:47:40 -07:00
Girish Ramakrishnan
3b740a5651 make tests work 2016-05-23 20:41:00 -07:00
Girish Ramakrishnan
7eb202f19a test: use the test-app instead of duplicating the checks in the tests 2016-05-23 20:17:11 -07:00
Girish Ramakrishnan
57888659a6 Update test app version 2016-05-23 18:31:12 -07:00
Girish Ramakrishnan
ebdefa7f18 test: oauth addon 2016-05-23 17:34:25 -07:00
Girish Ramakrishnan
6ccb806628 make apps-test pass 2016-05-23 16:31:02 -07:00
Girish Ramakrishnan
ae807b28b6 test: let server start the infra
otherwise, deps like dkim keys need to be setup in tests as well
2016-05-23 15:53:51 -07:00
Girish Ramakrishnan
00726b01e2 pass -no-run-if-empty instead 2016-05-23 15:50:36 -07:00
Girish Ramakrishnan
f5b777ab33 add route tests for username 2016-05-23 15:00:21 -07:00
Girish Ramakrishnan
10e0cbcebc do not set allowHalfOpen (otherwise we have to end socket ourself) 2016-05-23 10:50:04 -07:00
Girish Ramakrishnan
37512c4cac Wrap the stdin stream to indicate EOF
The docker exec protocol supports half-closing to signal that the stdin
is finished. The CLI tool tried to do this by closing it's half of the
socket when the stdin finished. Unfortunately, this does not work because
nginx immediately terminates a half-close :/ Node itself has no problem.

http://mailman.nginx.org/pipermail/nginx/2008-September/007388.html
seems to support the hypothesis. Basically, for HTTP and websockets
there is no notion of half-close.

Websocket protocol itself has no half-close as well:
http://www.lenholgate.com/blog/2011/07/websockets---i-miss-the-tcp-half-close.html
http://doc.akka.io/docs/akka/2.4.5/scala/http/client-side/websocket-support.html

The fix is to implement our own protocol that wrap stdin. We put a length
header for every payload. When we hit EOF, the length is set to 0. The server
sees this 0 length header and closes the exec container socket.
2016-05-22 22:27:49 -07:00
Girish Ramakrishnan
0aaaa866e4 Add a whole bunch of magic for docker.exec to work 2016-05-22 00:27:32 -07:00
Girish Ramakrishnan
53cb7fe687 debug out cmd 2016-05-19 15:54:35 -07:00
Girish Ramakrishnan
da42f2f00c fix boolean logic 2016-05-19 15:54:35 -07:00
Girish Ramakrishnan
efa3ccaffe fix crash because of missing error handling 2016-05-18 10:00:32 -07:00
Girish Ramakrishnan
e5d906a065 create fake cert/key for tests to pass 2016-05-13 22:35:13 -07:00
Girish Ramakrishnan
3ccb72f891 tests finally work 2016-05-13 22:05:05 -07:00
Girish Ramakrishnan
aa116ce58c fix tests 2016-05-13 20:21:09 -07:00
Girish Ramakrishnan
8e077660c4 start_addons is redundant 2016-05-13 17:57:56 -07:00
Girish Ramakrishnan
83a28afc8f addons are started by box code now 2016-05-09 23:56:02 -07:00
Girish Ramakrishnan
e76c7de259 bump test app version 2016-05-09 23:40:59 -07:00