search

smartdevicelink / protocol_spec

Describes the communication protocol between a smartdevicelink enabled head unit and mobile application
https://smartdevicelink.github.io/protocol_spec/
BSD 3-Clause "New" or "Revised" License
14 stars 13 forks source link

[SDL 0308] Add a Reason Parameter to All Protocol NAKs #31

Closed theresalech closed 4 years ago

theresalech commented 4 years ago

Proposal: Add a Reason Parameter to All Protocol NAKs

As a maintainer of an app library, it is often difficult to understand why Core has NAKed a protocol message. Most NAKs currently have a rejectedParams parameter, but in practice, this has not been enough information to determine why a NAK has occurred and properly alert the developer. In place of the rejectedParams parameter, the RegisterSecondaryTransportNAK has a reason parameter to describe the failure reason. This is a better approach and we should expand it to all NAK parameters.

Review: https://github.com/smartdevicelink/sdl_evolution/issues/1027

theresalech commented 4 years ago

Closed via #32