Improve broken security-events handling for private repositories without Advanced Security
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 50/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- github-actions, typescript
Piste de recherche
Commencez par la discussion de la pull request 2121 et l'issue 2117, puis comparez les sections de unknown-words.sh de check-spelling aux lignes 1122-1126 et 1610-1618. Suivez la gestion de security-events/SARIF par l'action et vérifiez que les dépôts privés sans Advanced Security reçoivent une réponse non fatale au lieu d'interrompre les workflows consommateurs.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
As noted in https://github.com/github/codeql-action/pull/2121#discussion_r1483012019 and https://github.com/github/codeql-action/issues/2117#issuecomment-1934863805, check-spelling has special code to handle the case where a user thinks they can use security-events: ... to talk to sarif reporting, but they're in a private repository and that repository doesn't have GitHub Advanced Security enabled:
https://github.com/check-spelling/check-spelling/blob/26b46adbdebd5dd0b34c7155113d50c40f43fb22/unknown-words.sh#L1610-L1618
https://github.com/check-spelling/check-spelling/blob/26b46adbdebd5dd0b34c7155113d50c40f43fb22/unknown-words.sh#L1122-L1126
Which results in this (more or less):
Check Spelling: .github/workflows/spelling.yml#L106
Unsupported configuration: use_sarif needs GitHub Advanced Security to be enabled - see https://docs.github.com/get-started/learning-about-github/about-github-advanced-security. (unsupported-configuration)
The codeql-action should have some code to handle this as well.
Ideally it'd do it in a way that is not fatal to actions/workflows that consume it.
check-spelling has to carefully decide not to call github/codeql-action today if it figures out that it'll fail, otherwise the action/workflow die because of this fault.
- Langage dominant
- TypeScript
- Étoiles
- 1.6k
- Forks
- 493
- Merge moyen
- 1 j 13 h
- PR mergées (30 j)
- 44
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de github/codeql-action
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
github/codeql-action#4052 · 4 commentaires ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
github/codeql-action#4078 · 1 commentaire ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
github/codeql-action#4008 · 9 commentaires ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 52/100
github/codeql-action#3978 · 4 commentaires · 1 réaction ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
github/codeql-action#3915 · 6 commentaires · 3 réactions ·
Toutes les issues de github/codeql-action
Issues similaires
-
blocklist removal
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
MetaMask/eth-phishing-detect#296544 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
pastelsky/bundlephobia#1122 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
category/development priority/P2 scope/file-operations scope/testing type/enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Enatega Customer and Rider app: Add-ons price is not visible to customer after order is placed. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100