Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • Q quickfix
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 48
    • Issues 48
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 25
    • Merge requests 25
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • quickfixgo
  • quickfix
  • Merge requests
  • !120

validator interface

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/bhaan/validator-interface into master Apr 26, 2016
  • Overview 5
  • Commits 2
  • Pipelines 0
  • Changes 4

Created by: bhaan

Related to #107 (closed)

The changes here propose a small refactor for message validation. I've created a validator interface, with two implementations, one for FIX, and another for FIXT. We currently have some logic in the session struct surrounding validation that determines which data dictionaries to use. With these changes, I've been able to move all of that decision making into the validation file.

Additionally, this refactor will encourage a more scalable approach for upcoming validation configuration enhancements, ex. #107 (closed)

Thoughts 💭 and concerns 😰 are more than welcome. Please and thank you! 😊

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/bhaan/validator-interface