mozilla-releng / mozilla-releng/scriptworker-scripts
[treescript] Stop using the `releng-treescript` app for Gecko l10n bumps
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 16
- Forks
- 38
- Ø Merge
- 1 T. 7 Std.
- Gemergte PRs (30 T.)
- 14
Beschreibung
We added the ability to store l10n changes in a separate Github repo for Gecko l10n bumps:
https://github.com/mozilla-releng/scriptworker-scripts/commit/3938c24da2c0964a1adc9994c3a8300df65ee0ae
We currently authenticate as the releng-treescript app to do things like get the latest commit and return the contents of files. This is useful for avoiding Github rate limits. However, we do not actually commit any changes to these external l10n repos. Therefore using releng-treescript, which has write access to a repo, does not follow the Principle of Least Privilege.
Instead, I believe we should create a generic releng-readonly (name TBD) app that only has read access to the repos. Then, we can install this app anywhere we simply want read access (e.g for higher rate limits or for access to private repos).
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der Überprüfung des referenzierten Commits und der aktuellen Authentifizierung, die für Gecko-l10n-Bumps und externe l10n-Repositories verwendet wird. Ermittle, wo Zugriff mit Leseberechtigung ohne Commit-Zugriff benötigt wird. Erledigt ist die Aufgabe, wenn der Zugriff auf Repositories mit Leseberechtigung eine separate App statt releng-treescript verwendet und der App-Name sowie der Installationsumfang geklärt sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- github, python
- Bereich
- authorization, security
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100