Skip to content

VALIDATORS can leak implementation details about what function handles a validator #467

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Julian opened this issue Sep 30, 2018 · 0 comments
Open

VALIDATORS can leak implementation details about what function handles a validator #467

Julian opened this issue Sep 30, 2018 · 0 comments
Labels
Bug Something doesn't work the way it should. Dialects v2 Issues which will likely be addressed as part of reworked dialect support

Comments

@Julian
Copy link
Member

Julian commented Sep 30, 2018

SomeValidator.VALIDATORS assumes a 1-1 mapping between validators and validating functions.

That's not necessarily the case though.

E.g., the implementation of the then validator happens inside if.
@Julian Julian added the Bug Something doesn't work the way it should. label Sep 30, 2018
Julian added a commit that referenced this issue Apr 29, 2022
@Julian
Squashed 'json/' changes from 3731ed32a..060caae0d
19d2c1a 
060caae0d Merge pull request #467 from gene-hightower/strict-rfc-grammar
995932c74 Additional RFC-5321 Mailbox tests
637f0ac3e Merge pull request #542 from jdesrosiers/dyanmicref-no-bookending
87944e52e Merge pull request #543 from jdesrosiers/update-vocabulary-for-draft-next
07b90e3b5 Merge pull request #544 from frawa/list-typed-json
365349adc list typed-json in new Scala section
9d1efc27e Update vocabulary tests for draft-next
89f59caad Update dynamicRef tests for bookenending removal

git-subtree-dir: json
git-subtree-split: 060caae0dd58e34af0449baec1103606a0ef4977
@Julian Julian added the Dialects v2 Issues which will likely be addressed as part of reworked dialect support label Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Something doesn't work the way it should. Dialects v2 Issues which will likely be addressed as part of reworked dialect support
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Julian