Skip to content

Nachweis eines neuen Pins enthält die ganze Workflow-Datei #35

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Musiker15 merged 1 commit into from
Oct 10, 2026
Merged

Nachweis eines neuen Pins enthält die ganze Workflow-Datei #35

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 8 additions & 3 deletions .github/workflows/security-review.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -235,6 +235,12 @@ jobs:
continue
on_main = git('merge-base', '--is-ancestor', sha, 'origin/main').returncode == 0
lines.append(f'- Liegt auf `main`: {"ja" if on_main else "nein"}')
# The whole file as well: a diff alone does not show where the
# lines it touches take their values from.
whole = f'pinned-review-workflow-{sha[:12]}.yml'
with open(os.path.join(review, whole), 'w', encoding='utf-8') as handle:
handle.write(git('show', f'{sha}:{workflow}').stdout)
lines.append(f'- `{workflow}` in diesem Commit, vollständig: `.security-review/{whole}`')
if base:
added = git('log', '--format=%h %s', f'{base}..{sha}').stdout.strip()
dropped = git('log', '--format=%h %s', f'{sha}..{base}').stdout.strip()
Expand All @@ -248,8 +254,6 @@ jobs:
print(f'Neuer Pin {sha[:7]}: auf main {"ja" if on_main else "nein"}, '
f'{len(added.splitlines())} Commits seit {base[:7]}.')
else:
content = git('show', f'{sha}:{workflow}').stdout
lines += ['', f'### `{workflow}` in diesem Commit', '', '```yaml', content.rstrip(), '```']
print(f'Neuer Pin {sha[:7]}: auf main {"ja" if on_main else "nein"}, ohne bisherigen Pin.')

with open(os.path.join(review, 'pinned-review-workflow.md'), 'w', encoding='utf-8') as handle:
Expand Down Expand Up @@ -309,7 +313,8 @@ jobs:
Reviews (`opengewerk/.github/.github/workflows/security-review.yml@<sha>`), steht in
`.security-review/pinned-review-workflow.md`, was der Workflow vor dir aus dem
öffentlichen Repository geholt hat: ob der Commit auf dessen `main` liegt, die Commits
dazwischen und der Diff der Workflow-Datei. Prüfe diesen Diff wie Code des PRs; ein
dazwischen, der Diff der Workflow-Datei und der Pfad, unter dem die ganze Datei dieses
Commits liegt. Prüfe den Diff im Zusammenhang der ganzen Datei wie Code des PRs; ein
Commit, der nicht auf `main` liegt oder sich nicht holen lässt, ist ein Befund.
2. Gib den Report als deine letzte Antwort aus, als Markdown auf Deutsch. Lege dafür keine
Datei an; ein Werkzeug zum Schreiben hast du nicht, und gelesen wird nur innerhalb des
Expand Down
Loading