Skip to content

I think the validation for retries/retryInterval (updatefirmware/getdiagnostics) may be incorrect #362

@ChairmanMawh

Description

@ChairmanMawh

(OCPPJ1.6)

Hi All

I have a software that uses CTEU to parse its messages and I'm getting a validation error when sending zero for retries or retryInterval in an update firmware. I think other messages may be validated similarly.

To me sending 0 for "retries" makes sense because I want to tell the charge point "try only one time; do not try again (retry) after a fail"

I think it also useful to point out that "retries" is in the errata 4.0 section 3.50 where the description was changed from "how many times the charge point must TRY to upload.." to ".. must RETRY to upload.."; in the old description it does seem like 1 would be a minimum value but now the latest specification has a description that agrees with the property name, that "retry" is "the attempt(s) after the first" and 0 would be a logical minimum

For "retryInterval" I would say that 0 means "you can try again immediately, no need to wait even 1 second" and we should allow 0 there

What do you think?
Regards
Matt

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions