Skip to content

SubscribeRequest prefix is required against specification #3

@remingtonc

Description

@remingtonc

gnmi/client/query.go

Lines 235 to 237 in f490526

if s.Subscribe.Prefix == nil {
return q, errors.New("Prefix field in SubscriptionList is nil")
}

SubscribeRequest implementation requires prefix despite language such as "The default prefix is null." and "This field MUST only ever be present on prefix paths in the corresponding request and response messages. This field is optional for clients."

Foolishly did not copy comment from fork (re-created this fork due to being messy), but previously there was commentary by maintainers stating that this was largely to accommodate a collector implementation vs a client implementation so that a collector would not attempt to collect for all potential targets. It appears there is, informally, a separation between a collector versus a client which has tainted some of the client aspects.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions