openservicebrokerapi / servicebroker

Open Service Broker API Specification
https://openservicebrokerapi.org/
Apache License 2.0
1.19k stars 434 forks source link

`client update` command fails #761

Closed kanngiesser closed 2 months ago

kanngiesser commented 2 months ago

What is the problem? The client update command fails with error message error deciding update path: passed maintenance_info does not match the catalog maintenance_info.

Who does this affect? service broker authors

Do you have any proposed solutions?

The client update should send a value for previous_values.maintenance_info.version and maintenance_info.version when sending the update request. Both values must match. With the current implementation, both previous_values and maintenance_info are left empty when sending the request.

Alternatively, brokerapi/broker/decider/decider.go should be updated to match the use-case for the client-update command instead of throwing ErrMaintenanceInfoConflict

Additional context Add any other context or screenshots can go here.

kanngiesser commented 2 months ago

I´m terribly sorry, I posted this issue to the wrong project