JakeHillion/storj
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

satellite/orders: don't version check old endpoint

Browse Source 
nodes are submitting using both the legacy and windowed endpoints
and thus having their legacy submissions rejected.

it is legal to use both the legacy and windowed endpoints
in phase1 since they use the same backend. the legacy endpoint
is disabled in phase2 and phase3.

therefore, if we wait an order expiration period (2 days) after
we determine enough nodes have started using the windowed
endpoint, we can be sure that any orders they did have to
submit with the legacy endpoint will have expired.

Change-Id: I4418a881bf8bb9377efaef4c651e6103a5dc6ed0
This commit is contained in:
Jeff Wendling 2020-09-09 10:23:46 -04:00
parent 2cf12b9409
commit 1fecaed7df
1 changed files with 0 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
satellite/orders/endpoint.go
View File

@ -255,13 +255,6 @@ func (endpoint *Endpoint) Settlement(stream pb.DRPCOrders_SettlementStream) (err
return rpcstatus.Error(rpcstatus.Unauthenticated, err.Error())
}
ok, err := endpoint.nodeAPIVersionDB.VersionAtLeast(ctx, peer.ID, nodeapiversion.HasWindowedOrders)
if err != nil {
return rpcstatus.Wrap(rpcstatus.Internal, err)
} else if ok {
return rpcstatus.Error(rpcstatus.PermissionDenied, "node api version too new")
}
formatError := func(err error) error {
if errors.Is(err, io.EOF) {
return nil