Skip to content
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

Clarify RecordingJobStateChange data #3

Closed
willysagefalk opened this issue Aug 28, 2020 · 1 comment
Closed

Clarify RecordingJobStateChange data #3

willysagefalk opened this issue Aug 28, 2020 · 1 comment
Labels
Minor Recording Control
Milestone
Change Request

Comments

@willysagefalk
Copy link
Member

willysagefalk commented Aug 28, 2020 •
edited by HansBusch
Loading

#2701

Current Problem
The DTT tests that the payload does not contain Sources information which absolutely makes sense to limit the size of the event.
The specification specifies that other information shall be provided .
Proposal
Add to 5.25.1
The device shall omit the Sources parameter when emitting the event.
Attachments (0)
Oldest first Newest first
Comments only

Change History (1)

Changed 4 months ago by hans.busch
Correction, our sample forgot to repeat the State information in the ElementItem. The DTT was just creating wrong output.
My conclusion is that the whole element item should be made optional since the only mandatory items state and token repeat the simple item parameters.
Note that this CR would probably require a relaxation of the DTT.
@willysagefalk willysagefalk added this to the Change Request milestone Aug 28, 2020
HansBusch added a commit that referenced this issue Sep 10, 2020
@HansBusch
Related to #3: Make potentially large element item optional.
ea2116f 
Additionally fix wrong conversion from word of source listing of next section.
@HansBusch
Copy link
Member

Created pull request #24.

HansBusch added a commit that referenced this issue Sep 11, 2020
@HansBusch
Revert "Related to #3: Make potentially large element item optional."
baec221 
This reverts commit ea2116f.
@HansBusch HansBusch mentioned this issue Sep 11, 2020
Merged
@jmelancongen jmelancongen mentioned this issue Aug 29, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Minor Recording Control
Projects
None yet
Milestone
Change Request
Development

No branches or pull requests
2 participants
@HansBusch @willysagefalk