AbsaOSS / AbsaOSS/ABRiS

Revert pull_request action back

Offen
#337 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Scala
Sterne
242
Forks
84
Ø Merge
10 Std. 12 Min.
Gemergte PRs (30 T.)
2

Beschreibung

This issue is related to:
- #333
- #334

Status in time of writing this issue is that we are using `pull_request_target` instead of usage `pull_request` in yml files:
- build-scala2.12-spark3.2.yml
- build-scala2.13-spark3.2.yml

This change was needed to be able to merge Approved changes from repository fork.
We were not able to find any better solution in time of change.

Now, when merge is done we are in state where another changes can come inside repository from forks.
It open potential security risk which was solved by simple change configuration for now.
- Configuration change: PR from fork will start GH actions after team member confirmation. This can avoid start of potential dangerous GH action with writing rights.

Fast fix:
- Return back `pull_request` action to both yml files mentioned above.

Slow fix:
- Invite DevOps colleagues into problem solution and do analysis what changes needs to be done.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.