It actually requires make uninstall && make clean && make && make install to 
ensure that the changed code is used.

Regards,
John Ralls

> On Mar 18, 2020, at 2:52 PM, Martin Preuss <[email protected]> wrote:
> 
> Hi,
> 
> okay now, the latest GIT version always sets "OFXHEADER:100" and uses
> the given header version only for the "VERSION:" header. In that case
> you can specify "102" and it should create the correct headers...
> 
> Please note: For reasons not yet understood it might be necessary when
> updating from git to first issue "make clean" before "make", otherwise
> sometimes the resulting binaries might not be based on all the latest
> code (I have no idea whether this happens for others as well, it's just
> something I stumbled upon while working on AqBanking).
> 
> 
> Regards
> Martin
> 
> 
> Am 18.03.20 um 22:47 schrieb John Ralls:
>> It won't let you leave that blank, and even if it did it would put '100' in 
>> both fields:
>> if (!(s && *s))
>>    s="100";
>> 
>>  GWEN_Buffer_AppendString(buf, "OFXHEADER:");
>>  GWEN_Buffer_AppendString(buf, s);
>>  GWEN_Buffer_AppendString(buf, "\r\nDATA:OFXSGML\r\n");
>> 
>>  GWEN_Buffer_AppendString(buf, "VERSION:");
>>  s=AO_User_GetHeaderVer(u);
>>  if (!s || !*s)
>>    s="102";
>>  GWEN_Buffer_AppendString(buf, s);
>>  GWEN_Buffer_AppendString(buf, "\r\n");
>> 
>> Because at the second test s has a value.
>> 
>> The new code doesn't emit the trailing tags, so that may not have been the 
>> problem. The main difference other than the OFXHEADER: value now appears to 
>> be that there are newlines between the fields. Are those getting escaped in 
>> GWEN_SyncIo_Http_Write?
>> 
>> Regards,
>> John Ralls
>>> On Mar 18, 2020, at 2:34 PM, Martin Preuss <[email protected]> wrote:
>>> 
>>> Hi,
>>> 
>>> not ATM, but that can be arranged. However, how about leaving that field
>>> empty in the settings dialog? That should default to 100 for OFXHEADER
>>> and 102 for VERSION (for - ahem - historic reasons, I'm sure I had
>>> good^H^H^H reasons for those mixed defaults... :-})
>>> 
>>> Regards
>>> Martin
>>> 
>>> 
>>> Am 18.03.20 um 22:26 schrieb Chris Graves:
>>>> Nice!  I see that the change has been made, but still receive the HTTP
>>>> 400 error.
>>>> 
>>>> In the AQB5 ofx.log file for a successful case, I see
>>>> OFXHEADER:100
>>>> VERSION:102
>>>> 
>>>> In the AQB6 file, I see
>>>> OFXHEADER:102
>>>> VERSION:102
>>>> 
>>>> Not sure if this could be the problem.  Is there a way to set OFXHEADER
>>>> and VERSION independently?
>>>> 
>>>> 
>>>> On Wed, Mar 18, 2020 at 2:02 PM Martin Preuss <[email protected]
>>>> <mailto:[email protected]>> wrote:
>>>> 
>>>>   Hi,
>>>> 
>>>>   you can change the header manually to 102/103 in the homebanking setup
>>>>   dialog (select user, edit user, switch to app settings and type-in
>>>>   header version 102).
>>>> 
>>>>   Regards
>>>>   Martin
>>>> 
>>>> 
>>>>   Am 18.03.20 um 21:57 schrieb Chris Graves:
>>>>> Hi Martin,
>>>>> 
>>>>> Looking better!  However, to my knowledge, my bank only supports OFX
>>>>> version 102 or 103.  I had been using 102.
>>>>> 
>>>>> ofx.log:
>>>>> Sending:
>>>>> -------------------------------------
>>>>> OFXHEADER:220
>>>>> DATA:OFXSGML
>>>>> VERSION:220
>>>>> SECURITY:NONE
>>>>> ENCODING:USASCII
>>>>> CHARSET:1252
>>>>> COMPRESSION:NONE
>>>>> OLDFILEUID:NONE
>>>>> NEWFILEUID:20200318135033.000
>>>>> 
>>>>> <OFX>
>>>>> <SIGNONMSGSRQV1>
>>>>> <SONRQ>
>>>>> <DTCLIENT>20200318135025.000
>>>>> 
>>>>> On Wed, Mar 18, 2020 at 1:32 PM Martin Preuss <[email protected]
>>>>   <mailto:[email protected]>
>>>>> <mailto:[email protected] <mailto:[email protected]>>> wrote:
>>>>> 
>>>>>     Hi Chris,
>>>>> 
>>>>>     could you please test the latest GIT version? It should create
>>>>   valid
>>>>>     OFXv1 requests now, however, I can't test it completely since
>>>>   I only
>>>>>     have access to an OFXv2 server...
>>>>> 
>>>>> 
>>>>>     Regards
>>>>>     Martin
>>>>> 
>>>>> 
>>>>>     Am 18.03.20 um 16:31 schrieb chris graves:
>>>>>> An additional bit of information from /tmp/ofx.log:
>>>>>> AQB5:
>>>>>> Sending:
>>>>>> -------------------------------------
>>>>>> OFXHEADER:100
>>>>>> DATA:OFXSGML
>>>>>> VERSION:102
>>>>>> SECURITY:NONE
>>>>>> ENCODING:USASCII
>>>>>> CHARSET:1252
>>>>>> COMPRESSION:NONE
>>>>>> OLDFILEUID:NONE
>>>>>> NEWFILEUID:20200318081757.000
>>>>>> 
>>>>>> <OFX>...
>>>>>     [...]
>>>>> 
>>>>> 
>>>>>     --
>>>>>     "Things are only impossible until they're not"
>>>>> 
>>>> 
>>>> 
>>>>   -- 
>>>>   "Things are only impossible until they're not"
>>>> 
>>> 
>>> 
>>> -- 
>>> "Things are only impossible until they're not"
>> 
> 
> 
> -- 
> "Things are only impossible until they're not"

_______________________________________________
gnucash-user mailing list
[email protected]
To update your subscription preferences or to unsubscribe:
https://lists.gnucash.org/mailman/listinfo/gnucash-user
If you are using Nabble or Gmane, please see 
https://wiki.gnucash.org/wiki/Mailing_Lists for more information.
-----
Please remember to CC this list on all your replies.
You can do this by using Reply-To-List or Reply-All.

Reply via email to