Skip to content

Rebase with main branch#172

Open
yuvaramachandran-gurusamy wants to merge 5 commits intotopic/RDKEMW-5927from
main
Open

Rebase with main branch#172
yuvaramachandran-gurusamy wants to merge 5 commits intotopic/RDKEMW-5927from
main

Conversation

@yuvaramachandran-gurusamy
Copy link
Contributor

No description provided.

…p_set_friendlyname

Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
RDKTV-35185: Add sync between ssdp_http_server_callback and gdial_ssdp_set_friendlyname
@yuvaramachandran-gurusamy yuvaramachandran-gurusamy requested a review from a team as a code owner September 25, 2025 18:09
* RDKECMF-213 Add CODEOWNERS file
* RDKEMW-254: libsoup3 support
* RDKTV-35185: Add sync between ssdp_http_server_callback and gdial_ssdp_set_friendlyname
* RDKEMW-2033: Fix coverity related issue
* RDK-55044: [RDKE] Implement DIAL requirement to use on EU product
* RDK-56672: Fix parsing issue while restrict addtionalData url per app
* RDKEMW-2278: Removal of WPEFrameworkSecurity Agent Utility
* RDKEMW-4129: Prepare native build environment
* RDKEMW-4129: Test
* RDKEMW-4129: Prepare native build script
* remmove duplicates from build_xxx script
* Update gdial.cpp
* RDKEMW-5927: Crash fix during soup_uri_to_string API call (#163)
* RDKEMW-6891:Coverity errors fix for xdial (#166)
* RDKEMW-6891: Coverity errors fix for xdial
* Update gdial.cpp
Fixed review comments
* Deploy cla action

---------

Signed-off-by: apatel859 <amit_patel5@comcast.com>
Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
Signed-off-by: Thamim Razith <ThamimRazith_AbbasAli@comcast.com>
Co-authored-by: Simon Chung <simon.chung@rdkcentral.com>
Co-authored-by: Stephen Barrett <sbarre01@users.noreply.github.com>
Co-authored-by: apatel859 <amit_patel5@comcast.com>
Co-authored-by: apatel859 <48992974+apatel859@users.noreply.github.com>
Co-authored-by: hgfell683 <107510770+hgfell683@users.noreply.github.com>
Co-authored-by: tabbas651 <thamimrazith_abbasali@comcast.com>
Co-authored-by: ssitar583 <soundaryaa_sitaram@comcast.com>
Co-authored-by: mukesh972 <mukesh.m@tataelxsi.co.in>
Co-authored-by: rdkcmf <github@code.rdkcentral.com>
Co-authored-by: Alan Ryan <20208488+Alan-Ryan@users.noreply.github.com>
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Comment on lines +11 to +25
name: Build xdialserver component in github rdkcentral
runs-on: ubuntu-latest
container:
image: ghcr.io/rdkcentral/docker-rdk-ci:latest

steps:
- name: Checkout code
uses: actions/checkout@v3

- name: native build
run: |
sh -x build_dependencies.sh
sh -x cov_build.sh
env:
GITHUB_TOKEN: ${{ secrets.RDKCM_RDKE }}

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}

Copilot Autofix

AI 3 months ago

To fix this issue, add a permissions block specifying the minimal permissions needed for the workflow. The best practice is to start with contents: read, then add additional permissions as needed for the workflow functionality. If the workflow only checks out code and runs build scripts (as shown), then contents: read should be sufficient. The permissions block may be added either at the root level (to apply to all jobs), or within the individual job(s) if different jobs require different permissions; for this workflow with a single job, either location will work, but root level is simpler and will prevent future issues if more jobs are added. The file to edit is .github/workflows/native_full_build.yml, and the block should be added near the top (e.g., after the name and before or after on), with proper YAML indentation.

Suggested changeset 1
.github/workflows/native_full_build.yml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/native_full_build.yml b/.github/workflows/native_full_build.yml
--- a/.github/workflows/native_full_build.yml
+++ b/.github/workflows/native_full_build.yml
@@ -1,5 +1,8 @@
 name: Build Component in Native Environment
 
+permissions:
+  contents: read
+
 on:
   push:
     branches: [ main, 'sprint/**', 'release/**', topic/*, develop ]
EOF
@@ -1,5 +1,8 @@
name: Build Component in Native Environment

permissions:
contents: read

on:
push:
branches: [ main, 'sprint/**', 'release/**', topic/*, develop ]
Copilot is powered by AI and may make mistakes. Always verify output.
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.

4 participants