5G-MAG / rt-5gms-application-function

5G Media Streaming - Application Function
https://www.5g-mag.com/streaming
Other
11 stars 6 forks source link

QoE Metrics Reporting - M1 Provisioning - reportingInterval shall be greater than zero #145

Closed dsilhavy closed 8 months ago

dsilhavy commented 8 months ago

Description

According to TS 26.512 Table 7.8.3-1 the reportingInterval shall be greater than zero:

The time interval between successive metrics reports. The value shall be greater than zero. If not specified, a single final report shall be sent after the media streaming session has ended.

The current implementation accepts values equal and below zero. Resulting output when reading the configuration afterwards

{
    "reportingInterval": -1,
}
rjb1000 commented 8 months ago

Looks like this will need to be validated manually (at least for the time being) since the underlying YAML data type for DurationSec is a signed integer without a sensible minimum value of 0.

rjb1000 commented 8 months ago

See also #146.

dsilhavy commented 8 months ago

Confirmed this is fixed with the latest changes