plotly / plotly/dash-ag-grid

Asks for updates to this package's repository security.

Open
#337 6 comments 4 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

community feature P2 security
Dominant language
Python
Stars
241
Forks
47
PR merge metrics
No merged PRs in 30d

Description

Hi, our project utilizes a lot of dash plotly packages (really appreciate all your work!), and would like to leverage dash-ag-grid for some new functionalities under design/development.
However, we are concerned about the security setup of this repository, and the risk of future bad changes making into the package.
We used the tool https://github.com/ossf/scorecard to help us assess the repository security.
Some of the major concerning areas are:

  1. branch protection - the 'main' branch is not under any branch protection rule that governs write access and how changes make into releases. The recommendation is https://github.com/ossf/scorecard/blob/main/docs/checks.md#branch-protection.
  2. token permission -
    Warn: jobLevel 'contents' permission set to 'write': .github/workflows/release.yml:13
    Warn: no topLevel permission defined: .github/workflows/python-test.yml:1
    Warn: no topLevel permission defined: .github/workflows/release.yml:1
    Which can be easily mitigated, see https://github.com/ossf/scorecard/blob/main/docs/checks.md#token-permissions.

Can you let me know if those security configurations can be updated soon? As it is, we would like to use the dash-ag-grid but cannot due to the security concerns (given the rise of software pipeline attacks).

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Review the repository's main branch settings and .github/workflows/release.yml and .github/workflows/python-test.yml, then compare them with the referenced OSS-Fuzz Scorecard branch-protection and token-permissions guidance. Done means the requested branch protection and least-privilege workflow permissions are configured and the security assessment no longer reports these warnings.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions
Domain
ci-cd, security
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.