smartdevicelink / sdl_core

SmartDeviceLink In-Vehicle Software and Sample HMI
BSD 3-Clause "New" or "Revised" License
240 stars 245 forks source link

SDL requests invalid VR.AddCommand during CreateInteractionChoiceSet resumption without VR part #3888

Closed GetmanetsIrina closed 2 years ago

GetmanetsIrina commented 2 years ago

Bug Report

SDL requests invalid VR.AddCommand during CreateInteractionChoice resumption without VR part

Precondition
  1. SDL and HMI are started
  2. Mobile app is registered and activated
Reproduction Steps
  1. Mobile app requests CreateInteractionChoiceSet without VR Command
  2. Perform transport reconnect
  3. Mobile app is registered with actual hash id
Expected Behavior

SDL does not request VR.AddCommand for the CreateInteractionChoiceSet resumption without VR Command and resumes CreateInteractionChoiceSet internally

Observed Behavior

SDL request invalid VR.AddCommand message during resumption

SDL -> HMI [16:36:20:329]: {"id":96,"jsonrpc":"2.0","method":"VR.AddCommand","params":{"appID":856255661,"cmdID":4,"grammarID":0,"type":"Choice","vrCommands":null}}

VRRPC.js:295 FFW.VR.AddCommandResponse
RPCClient.js:323 HMI -> SDL [16:36:20:330]: {"jsonrpc":"2.0","id":96,"error":{"code":11,"message":"Parameter 'vrCommands' does not exists!","data":{"method":"VR.AddCommand"}}}
Note:

In case of deleting such CreateInteractionChoiceSet(without VR part) after it adding ( step 1 from ) SDL also requests invalid VR.DeleteCommand

{"id":70,"jsonrpc":"2.0","method":"VR.DeleteCommand","params":{"appID":1326361392,"cmdID":1,"grammarID":null,"type":"Choice"}}
OS & Version Information
Logs

SmartDeviceLinkCore.zip

GetmanetsIrina commented 2 years ago

Please notice this is a non-regression issue. It's also reproduced on release 8.0

LitvinenkoIra commented 2 years ago

Luxoft team is going to fix this issue

ShobhitAd commented 2 years ago

Closed via https://github.com/smartdevicelink/sdl_core/pull/3938