Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CheckButtonName check not working for module types other than CLIMATE and RADIO #3945

Closed
ShobhitAd opened this issue Aug 31, 2022 · 1 comment
Projects

Comments

@ShobhitAd
Copy link
Contributor

Bug Report

Currently, core only checks if a button name is compatible with the module type (in CheckButtonName) if the module type in the request is CLIMATE or RADIO.

If the user were to send a ButtonPress request with a different type, core would currently not send the "Request module type and button name mismatch!" response and forward the invalid request to the HMI

Reproduction Steps
  1. Add RemoteControl permissions in preloaded_pt
"app_policies": {
            "default": {
                "keep_context": false,
                "steal_focus": false,
                "priority": "NONE",
                "default_hmi": "NONE",
                "groups": [
                    "Base-4",
+                    "RemoteControl"
                ],
+                "moduleType": [],
                "RequestType": [],
                "RequestSubType": []
            },
  1. Start core and HMI
  2. Register and activate RPC Builder app with appHMIType REMOTE_CONTROL
  3. Send ButtonPress with params {"moduleType":"HMI_SETTINGS","buttonName":"FAN_UP","buttonPressMode":"LONG"}
Expected Behavior

App receives INVALID_DATA response with info "Request module type and button name mismatch!"

Observed Behavior

App receives SUCCESS response

OS & Version Information
  • OS/Version: Ubuntu 20.04
  • SDL Core Version: release/8.2.0
  • Testing Against: RPC Builder
@theresalech theresalech added this to Development In Progress/PR Ready for Review in 8.2.0 Aug 31, 2022
@theresalech theresalech moved this from Development In Progress/PR Ready for Review to Review In Progress in 8.2.0 Sep 6, 2022
@jacobkeeler
Copy link
Contributor

Closed via #3944

@jacobkeeler jacobkeeler moved this from Review In Progress to Completed Bug Fixes/ Enhancements in 8.2.0 Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
8.2.0
Completed Bug Fixes / Enhancements
Development

No branches or pull requests

2 participants