Thanks! This was one of the longest awaited fixes. Funny that it was just about a newline.
Perhaps the bugfix can go to Dec2016?

Best, Roberto

On 31 May 2017 at 13:55, Panagiotis Koutsourakis <commits+kutsurak=monetdbsolutions.com@monetdb.org> wrote:
Changeset: e00fb765660d for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=e00fb765660d
Modified Files:
        tools/merovingian/daemon/controlrunner.c
Branch: Jul2017
Log Message:

Daemon was sending wrong answer for authorization request

When the password was correct, the server would send one extra new line that
confused the client. This commit fixes bug 3545


diffs (12 lines):

diff --git a/tools/merovingian/daemon/controlrunner.c b/tools/merovingian/daemon/controlrunner.c
--- a/tools/merovingian/daemon/controlrunner.c
+++ b/tools/merovingian/daemon/controlrunner.c
@@ -169,7 +169,7 @@ control_authorise(
        }
        free(pwd);

-       mnstr_printf(fout, "=OK\n");
+       mnstr_printf(fout, "=OK");
        mnstr_flush(fout);

        return 1;
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list