<div dir="ltr">Thanks!</div><div class="gmail_extra"><br><div class="gmail_quote">On 1 June 2017 at 10:24, Panagiotis Koutsourakis <span dir="ltr"><<a href="mailto:kutsurak@monetdbsolutions.com" target="_blank">kutsurak@monetdbsolutions.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Roberto,<br>
<br>
I just figured out a better implementation that would be backward<br>
compatible with any old server. I will revert this one and push the new<br>
one in the December branch, but It will probably not make it to the<br>
Dec2016-SP5 release.<br>
<br>
Best regards,<br>
Panos.<br>
<br>
Roberto Cornacchia @ 2017-05-31 15:16 GMT:<br>
<div><div class="h5"><br>
> Thanks! This was one of the longest awaited fixes. Funny that it was just<br>
> about a newline.<br>
> Perhaps the bugfix can go to Dec2016?<br>
><br>
> Best, Roberto<br>
><br>
> On 31 May 2017 at 13:55, Panagiotis Koutsourakis <<br>
> commits+kutsurak=<a href="mailto:monetdbsolutions.com@monetdb.org">monetdbsoluti<wbr>ons.com@monetdb.org</a>> wrote:<br>
><br>
>> Changeset: e00fb765660d for MonetDB<br>
>> URL: <a href="http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=e00fb765660d" rel="noreferrer" target="_blank">http://dev.monetdb.org/hg/<wbr>MonetDB?cmd=changeset;node=<wbr>e00fb765660d</a><br>
>> Modified Files:<br>
>>         tools/merovingian/daemon/<wbr>controlrunner.c<br>
>> Branch: Jul2017<br>
>> Log Message:<br>
>><br>
>> Daemon was sending wrong answer for authorization request<br>
>><br>
>> When the password was correct, the server would send one extra new line<br>
>> that<br>
>> confused the client. This commit fixes bug 3545<br>
>><br>
>><br>
>> diffs (12 lines):<br>
>><br>
>> diff --git a/tools/merovingian/daemon/<wbr>controlrunner.c<br>
>> b/tools/merovingian/daemon/<wbr>controlrunner.c<br>
>> --- a/tools/merovingian/daemon/<wbr>controlrunner.c<br>
>> +++ b/tools/merovingian/daemon/<wbr>controlrunner.c<br>
>> @@ -169,7 +169,7 @@ control_authorise(<br>
>>         }<br>
>>         free(pwd);<br>
>><br>
>> -       mnstr_printf(fout, "=OK\n");<br>
>> +       mnstr_printf(fout, "=OK");<br>
>>         mnstr_flush(fout);<br>
>><br>
>>         return 1;<br>
>> ______________________________<wbr>_________________<br>
>> checkin-list mailing list<br>
>> <a href="mailto:checkin-list@monetdb.org">checkin-list@monetdb.org</a><br>
>> <a href="https://www.monetdb.org/mailman/listinfo/checkin-list" rel="noreferrer" target="_blank">https://www.monetdb.org/<wbr>mailman/listinfo/checkin-list</a><br>
>><br>
</div></div>> ______________________________<wbr>_________________<br>
> developers-list mailing list<br>
> <a href="mailto:developers-list@monetdb.org">developers-list@monetdb.org</a><br>
> <a href="https://www.monetdb.org/mailman/listinfo/developers-list" rel="noreferrer" target="_blank">https://www.monetdb.org/<wbr>mailman/listinfo/developers-<wbr>list</a><br>
<br>______________________________<wbr>_________________<br>
developers-list mailing list<br>
<a href="mailto:developers-list@monetdb.org">developers-list@monetdb.org</a><br>
<a href="https://www.monetdb.org/mailman/listinfo/developers-list" rel="noreferrer" target="_blank">https://www.monetdb.org/<wbr>mailman/listinfo/developers-<wbr>list</a><br>
<br></blockquote></div><br></div>