Repository navigation
Security-Review belegt ein angehobenes Submodul - #45
Merged
Merged
Conversation
…em Repository Hebt ein Pull Request ein Submodul an, steht im Diff nur die Zeile "Subproject commit". Ein neuer Schritt vor dem Agenten holt ohne Zugangsdaten aus dem Repository des Submoduls, ob der neue Commit auf dessen Standardzweig liegt, die Commits dazwischen, die geänderten Dateien und den Diff, und legt sie unter .security-review/ ab. Adressen außerhalb der Organisation werden nicht geholt und als Befund gemeldet. Closes #41
This was referenced Oct 20, 2026
Musiker15
added a commit
to OpenGewerk/opengewerk
that referenced
this pull request
Oct 20, 2026
Hebt den angehefteten Commit des Security-Reviews von `c12e3c2` auf `731e352` in opengewerk/.github an. Dazwischen ändert sich am Workflow nur OpenGewerk/.github#45: hebt ein Pull Request ein Submodul an, holt ein Schritt vor dem Agenten ohne Zugangsdaten den Diff des Submoduls aus dessen öffentlichem Repository und legt ihn unter `.security-review/` ab, statt das Submodul offen zu lassen (OpenGewerk/.github#41). Die übrigen Commits dazwischen betreffen nur das Profil der Organisation. Den neuen Pin belegt das Review selbst im Schritt "Neuen Pin des Reviews belegen".
Musiker15
added a commit
to OpenGewerk/opengewerk-kanzlei
that referenced
this pull request
Oct 20, 2026
Hebt den angehefteten Commit des Security-Reviews von `c12e3c2` auf `731e352` in opengewerk/.github an. Dazwischen ändert sich am Workflow nur OpenGewerk/.github#45: hebt ein Pull Request ein Submodul an, holt ein Schritt vor dem Agenten ohne Zugangsdaten den Diff des Submoduls aus dessen öffentlichem Repository und legt ihn unter `.security-review/` ab, statt das Submodul offen zu lassen (OpenGewerk/.github#41). Die übrigen Commits dazwischen betreffen nur das Profil der Organisation. Den neuen Pin belegt das Review selbst im Schritt "Neuen Pin des Reviews belegen".
Musiker15
added a commit
to OpenGewerk/opengewerk-api-spec
that referenced
this pull request
Oct 20, 2026
Hebt den angehefteten Commit des Security-Reviews von `c12e3c2` auf `731e352` in opengewerk/.github an. Dazwischen ändert sich am Workflow nur OpenGewerk/.github#45: hebt ein Pull Request ein Submodul an, holt ein Schritt vor dem Agenten ohne Zugangsdaten den Diff des Submoduls aus dessen öffentlichem Repository und legt ihn unter `.security-review/` ab, statt das Submodul offen zu lassen (OpenGewerk/.github#41). Die übrigen Commits dazwischen betreffen nur das Profil der Organisation. Den neuen Pin belegt das Review selbst im Schritt "Neuen Pin des Reviews belegen".
Musiker15
added a commit
to OpenGewerk/opengewerk-kanzlei
that referenced
this pull request
Nov 6, 2026
Die Organisation heißt auf GitHub jetzt `OpenGewerk` statt `opengewerk`. Alle Links auf github.com und raw.githubusercontent.com zeigen auf den neuen Namen. GitHub leitet die alten Adressen weiter, es geht nur darum, dass nichts mehr über die Weiterleitung läuft. Verweise wie `OpenGewerk/.github#45` im CHANGELOG bleiben Text, sie sind in einer Datei keine Links. ## Eigene Festlegungen, die Moritz noch nicht gesehen hat - Der Aufruf des Security-Reviews wird nach dem Merge von OpenGewerk/.github#50 auf den neuen Commit angehoben, im selben Pull Request.
Musiker15
added a commit
to OpenGewerk/opengewerk-api-spec
that referenced
this pull request
Nov 6, 2026
Die Organisation heißt auf GitHub jetzt `OpenGewerk` statt `opengewerk`. Alle Links auf github.com und raw.githubusercontent.com zeigen auf den neuen Namen. GitHub leitet die alten Adressen weiter, es geht nur darum, dass nichts mehr über die Weiterleitung läuft. Verweise wie `OpenGewerk/.github#45` im CHANGELOG bleiben Text, sie sind in einer Datei keine Links. ## Eigene Festlegungen, die Moritz noch nicht gesehen hat - Der Aufruf des Security-Reviews wird nach dem Merge von OpenGewerk/.github#50 auf den neuen Commit angehoben, im selben Pull Request.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hebt ein Pull Request ein Submodul an, in
opengewerk-haustechnikdas Fundament unterupstream/opengewerk, steht indiff.patchnur die ZeileSubproject commit. Das Review konnte den Diff dazwischen in der Sandbox nicht holen und endete je nach Lauf mit "kein Befund" oder mitneeds_follow_upund rotem Status.Was sich ändert
Ein neuer Schritt "Angehobene Submodule belegen" läuft nach "Neuen Pin des Reviews belegen" und vor dem Agenten, nach demselben Muster:
diff.patchjedes Gitlink, dessen Commit sich ändert, und aus der.gitmodulesdes Heads die Adresse des Submoduls.https://github.com/opengewerk/<repo>, ohne Zugangsdaten und ohne Code aus dem PR auszuführen. Eine andere Adresse wird nicht geholt und steht als Befund in der Datei..security-review/submodules.mdstehen bisheriger und neuer Commit, ob der neue auf dem Standardzweig liegt, die Commits dazwischen, wegfallende Commits und die geänderten Dateien. Der ganze Diff des Submoduls liegt daneben alssubmodule-<n>-<sha>.diff, gekürzt erst ab 3 MB.Prüfung
Lokal mit dem Diff von OpenGewerk/opengewerk-haustechnik#489 (Submodul
1d01a3aauf57e099c): der Schritt findet das Gitlink, holt sieben Commits, die Liste der Dateien und einen Diff von knapp 1 MB. Mit einer fremden Adresse in.gitmodulesholt er nichts und schreibt den Befund. Ohne angehobenes Submodul endet er mit "Der PR hebt kein Submodul an."Closes #41