Skip to content

RDKEMW-17614 : mask_pii RFC not applied correctly - #218

Merged
dwolaver merged 1 commit into
developfrom
feature/RDKEMW-17614_mask_pii_rfc_update
Apr 27, 2026
Merged

RDKEMW-17614 : mask_pii RFC not applied correctly#218
dwolaver merged 1 commit into
developfrom
feature/RDKEMW-17614_mask_pii_rfc_update

Conversation

@dwolaver

Copy link
Copy Markdown
Contributor

No description provided.

@dwolaver
dwolaver requested a review from a team as a code owner April 24, 2026 16:28
Copilot AI review requested due to automatic review settings April 24, 2026 16:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes application of the mask_pii RFC value so that changes propagate beyond voice to other components that log keycodes, ensuring consistent PII masking behavior across the control manager.

Changes:

  • Apply RFC-updated mask_pii value to the voice session (guarded by null check).
  • Apply RFC-updated mask_pii value to the IR controller keycode masking.
  • Apply RFC-updated mask_pii value to all instantiated networks’ keycode masking.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@klu339 klu339 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved for QA test

@dwolaver
dwolaver merged commit de7fa44 into develop Apr 27, 2026
12 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants