Add protocol mapper sanity check to protodef-validator
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 65/100
- Issue type
- Feature
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- javascript
- Domain
- tooling
Research direction
Start by locating the protocol-switch validation logic and the tests for mapper and custom datatype handling. Add a sanity check for switch values when the preceding mapper's possible enum values are known, while preserving validation for custom datatypes where those values are unavailable; done means invalid references are reported and valid or unknowable cases continue to work.
Written by the indexing model from the issue text.
Description
Protocol switches could reference values absent from their preceding mapper, causing undefined dispatch behavior
There should be sanity check to ensure switches happen on enum values that are defined, if the interpreter/compiler knows what the possible enumerated values are (custom datatypes may not make this information available for sanity check)
Related
- Dominant language
- JavaScript
- Stars
- 1
- Forks
- 4
- Avg merge
- 2m
- Merged PRs (30d)
- 2
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ProtoDef-io/node-protodef-validator
-
Improve speed Open
Difficulty 5/5 Over a week Newbie friendliness 30/100
ProtoDef-io/node-protodef-validator#1 · 2 comments ·
All issues in ProtoDef-io/node-protodef-validator
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
mksglu/context-mode#1200 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
neondatabase/website#5944 ·
-
module: core
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
bigbluebutton/bigbluebutton#25849 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
jaegertracing/jaeger-ui#4506 ·