[QFJ-654] Rejecting due to OrigSendingTime not being in Tag without Data Dictionary Created: 24/Nov/11 Updated: 15/Nov/12 Resolved: 12/Dec/11 |
|
Status: | Closed |
Project: | QuickFIX/J |
Component/s: | Engine |
Affects Version/s: | 1.5.1 |
Fix Version/s: | 1.5.2 |
Type: | Improvement | Priority: | Default |
Reporter: | Christophe Domas | Assignee: | Christoph John |
Resolution: | Fixed | Votes: | 0 |
Labels: | session |
Issue Links: |
|
Description |
Hi, I'm using quickfix/J and for some reason I keep rejecting messages with reject: 20090210-16:13:01: Message 641 Rejected: Required tag missing:122 I have the following configuration options set in configuration: UseDataDictionary=N And I have also attempted to rip out OrigSendingTime from my Data Dictionary as a precautionary measure. I don't ever explicitly call this tag in code. Any help on this issue would be appreciated. Regards, |
Comments |
Comment by Christophe Domas [ 24/Nov/11 ] |
Hi, I've cloned the issue |
Comment by Grant Birchmeier [ 24/Nov/11 ] |
|
Comment by Christophe Domas [ 24/Nov/11 ] |
Yes, the problem is marked fixed and I use the last 1.5.1 version (that include the patch). |
Comment by Christophe Domas [ 28/Nov/11 ] |
The only way I found to intercept the message to add missing tag 122 is to set an IoFilterChainBuilder on SocketInitiator. Is there a better way to intercept FIX message before session handles it? |
Comment by Christophe Domas [ 28/Nov/11 ] |
Maybe I should create a new ticket: "How can I intercept quickfix message before session handles it", no? |
Comment by Christoph John [ 29/Nov/11 ] |
Hi guys, indeed, if (!header.isSetField(OrigSendingTime.FIELD)) { generateReject(msg, SessionRejectReason.REQUIRED_TAG_MISSING, OrigSendingTime.FIELD); return false; } |
Comment by Christoph John [ 30/Nov/11 ] |
Hi Christophe, |
Comment by Krzysztof Szalast [ 06/Sep/12 ] |
Hi, Maybe should be added property something like: P.S. Sorry for my english |
Comment by Christoph John [ 10/Sep/12 ] |
Cześć Krzysztof, |
Comment by Krzysztof Szalast [ 14/Sep/12 ] |
I agree. So I am waiting for new version. Thanks |