Skip to content

RDK-61609: Remove Sensitive StringFrom Open Source#40

Open
aruna007 wants to merge 1 commit into
developfrom
feature/RDK-61609
Open

RDK-61609: Remove Sensitive StringFrom Open Source#40
aruna007 wants to merge 1 commit into
developfrom
feature/RDK-61609

Conversation

@aruna007

Copy link
Copy Markdown

Reason for change: Remove Sensitive String (pattern036) From Open Source
Test Procedure: Tested the tasks locally
Risks: Low
Signed-off-by: aruna.glori@gmail.com

Reason for change: Remove Sensitive String (pattern036) From Open Source
Test Procedure: Tested the tasks locally
Risks: Low
Signed-off-by: aruna.glori@gmail.com
Copilot AI review requested due to automatic review settings June 11, 2026 10:52
@aruna007 aruna007 requested a review from a team as a code owner June 11, 2026 10:52
@github-actions

Copy link
Copy Markdown


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

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

Removes a sensitive string (pattern036) from the xhsScript.sh flow by renaming the temporary /tmp/xhs/... file path used to retrieve “Parameter2” via GetConfigFile.

Changes:

  • Replaced the previous obfuscated /tmp/xhs/cyavtfjzx.pse* filenames with clearer /tmp/xhs/xhs-param2* filenames.
  • Preserved the partner-specific behavior for cox by keeping a distinct temp file name suffix.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants