Skip to content

Commit

Permalink
browse: do not set timestamp for empty view
Browse files Browse the repository at this point in the history
To browse the full nodeset the client should use an empty (zero) ViewID
in the browse request. The server should probably ignore the timestamp
and/or version field but ABB 800xA does not return any nodes if the
ViewID is zero but the timestamp field is set.

This patch removes the timestamp field for the empty view.
  • Loading branch information
magiconair committed Oct 2, 2020
1 parent 2999796 commit 760b029
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions node.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ package opcua

import (
"strings"
"time"

"github.com/gopcua/opcua/id"
"github.com/gopcua/opcua/ua"
Expand Down Expand Up @@ -184,8 +183,7 @@ func (n *Node) References(refType uint32, dir ua.BrowseDirection, mask ua.NodeCl

req := &ua.BrowseRequest{
View: &ua.ViewDescription{
ViewID: ua.NewTwoByteNodeID(0),
Timestamp: time.Now(),
ViewID: ua.NewTwoByteNodeID(0),
},
RequestedMaxReferencesPerNode: 0,
NodesToBrowse: []*ua.BrowseDescription{desc},
Expand Down

0 comments on commit 760b029

Please sign in to comment.