[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 |