[QFJ-145] Provide a Session.validateMessageBody() method to only validate the body of the message without header or trailer parts. Created: 28/Feb/07 Updated: 25/Sep/08 Resolved: 28/Feb/07 |
|
Status: | Closed |
Project: | QuickFIX/J |
Component/s: | Engine |
Affects Version/s: | 1.0.5 |
Fix Version/s: | 1.1.0 |
Type: | Improvement | Priority: | Default |
Reporter: | Toli Kuznets | Assignee: | Toli Kuznets |
Resolution: | Fixed | Votes: | 0 |
Labels: | None |
Issue Links: |
|
Description |
What do you think of adding another method to quickfix.Session that allows to check if just the body of the message is valid? I can see the use cases being where a quickfix message is created (a new order single, a reject, etc) and may need to be validated in the application layer before sending it out on the wire (where it potentially gets validated as well by QFJ itself). However, it's hard to do that since the header and trailer information are injected into the message by QFJ. And while you can set almost anything, the CheckSum is hard to calculate; and it seems silly to duplicate the effort. Any thoughts? What do you guys think? |
Comments |
Comment by Toli Kuznets [ 28/Feb/07 ] |
Fix for bug 143 means that all the hand-constructed messages that have previously passed validation may no longer do that if they didn't have the header/trailer fields setup. |
Comment by Steve Bate [ 28/Feb/07 ] |
This sounds like a good idea. Did you want to go ahead and make the changes? Do you have permissions to assign issues? If not, let me know and I'll make the changes to Jira. |
Comment by Toli Kuznets [ 28/Feb/07 ] |
Overloaded the DadaDictionary.validate() message call to add a boolean to you can now call dictionary.validate(message, true) to validate just the body |
Comment by Toli Kuznets [ 28/Feb/07 ] |
checked into SVN in changeset 597 |