Repository navigation
Security-Review: Schlüsseldatei der Paketquelle darf nur den einen Schlüssel enthalten (#39) - #40
Merged
Merged
Conversation
…hlüssel enthalten (#39) Geprüft wurde der erste Fingerabdruck der Datei, apt vertraut aber jedem Schlüssel darin. Die Datei muss jetzt genau einen Hauptschlüssel enthalten, den erwarteten; sonst läuft das Review ohne Datenbank weiter.
MorlockNinja
pushed a commit
to opengewerk/opengewerk-haustechnik
that referenced
this pull request
Oct 12, 2026
Der Aufruf zeigt jetzt auf c12e3c2, den Commit von opengewerk/.github#40 auf dessen main. Dort muss die Schlüsseldatei der Paketquelle von PostgreSQL genau den einen erwarteten Schlüssel enthalten; geprüft wurde vorher nur der erste der Datei.
MorlockNinja
pushed a commit
to opengewerk/opengewerk
that referenced
this pull request
Oct 12, 2026
Der Aufruf zeigt jetzt auf c12e3c2, den Commit von opengewerk/.github#40 auf dessen main. Dort muss die Schlüsseldatei der Paketquelle von PostgreSQL genau den einen erwarteten Schlüssel enthalten; geprüft wurde vorher nur der erste der Datei.
MorlockNinja
pushed a commit
to opengewerk/opengewerk-kanzlei
that referenced
this pull request
Oct 12, 2026
Der Aufruf zeigt jetzt auf c12e3c2, den Commit von opengewerk/.github#40 auf dessen main. Dort muss die Schlüsseldatei der Paketquelle von PostgreSQL genau den einen erwarteten Schlüssel enthalten; geprüft wurde vorher nur der erste der Datei.
MorlockNinja
pushed a commit
to opengewerk/opengewerk-api-spec
that referenced
this pull request
Oct 12, 2026
Der Aufruf zeigt jetzt auf c12e3c2, den Commit von opengewerk/.github#40 auf dessen main. Dort muss die Schlüsseldatei der Paketquelle von PostgreSQL genau den einen erwarteten Schlüssel enthalten; geprüft wurde vorher nur der erste der Datei.
MorlockNinja
pushed a commit
to opengewerk/opengewerk-website
that referenced
this pull request
Oct 12, 2026
Der Aufruf zeigt jetzt auf c12e3c2, den Commit von opengewerk/.github#40 auf dessen main. Dort muss die Schlüsseldatei der Paketquelle von PostgreSQL genau den einen erwarteten Schlüssel enthalten; geprüft wurde vorher nur der erste der Datei.
Musiker15
added a commit
to opengewerk/opengewerk-website
that referenced
this pull request
Oct 12, 2026
* Security-Review: Aufruf auf opengewerk/.github#38 heben Der Aufruf zeigt jetzt auf fb4808a, den Commit von opengewerk/.github#38 auf dessen main. Eine Datenbank nennt er nicht. Das Review erreicht damit die Registry über den Proxy seiner Sandbox, findet pnpm im Pfad und installiert in einer Kopie des Checkouts. Vorher ließ sich in der Sandbox nichts installieren, und kein Test lief dort. * Security-Review: Aufruf auf opengewerk/.github#40 heben Der Aufruf zeigt jetzt auf c12e3c2, den Commit von opengewerk/.github#40 auf dessen main. Dort muss die Schlüsseldatei der Paketquelle von PostgreSQL genau den einen erwarteten Schlüssel enthalten; geprüft wurde vorher nur der erste der Datei.
Musiker15
added a commit
to opengewerk/opengewerk-api-spec
that referenced
this pull request
Oct 12, 2026
Hebt den Aufruf des Security-Reviews auf `c12e3c2`, den Commit von opengewerk/.github#40 auf dessen `main`. Er enthält opengewerk/.github#38 und die Korrektur aus #40. Das Review kann damit Tests ausführen. Eine Datenbank bekommt es hier nicht, weil der Aufruf keine nennt. Was sich am Workflow geändert hat und wie es geprüft wurde, steht in opengewerk/.github#38: Proxy der Sandbox für Node, `pnpm` im Pfad, eine Kopie des Checkouts zum Installieren, PostgreSQL in der Sandbox. Die Einstellungen der Sandbox sind unverändert. opengewerk/.github#40 verlangt dazu, dass die Schlüsseldatei der Paketquelle von PostgreSQL genau den einen erwarteten Schlüssel enthält. Teil von opengewerk/.github#37.
Musiker15
added a commit
to opengewerk/opengewerk-kanzlei
that referenced
this pull request
Oct 12, 2026
Hebt den Aufruf des Security-Reviews auf `c12e3c2`, den Commit von opengewerk/.github#40 auf dessen `main`. Er enthält opengewerk/.github#38 und die Korrektur aus #40. Das Review kann damit Tests ausführen. Eine Datenbank bekommt es hier nicht, weil der Aufruf keine nennt. Was sich am Workflow geändert hat und wie es geprüft wurde, steht in opengewerk/.github#38: Proxy der Sandbox für Node, `pnpm` im Pfad, eine Kopie des Checkouts zum Installieren, PostgreSQL in der Sandbox. Die Einstellungen der Sandbox sind unverändert. opengewerk/.github#40 verlangt dazu, dass die Schlüsseldatei der Paketquelle von PostgreSQL genau den einen erwarteten Schlüssel enthält. Teil von opengewerk/.github#37.
Musiker15
added a commit
to opengewerk/opengewerk
that referenced
this pull request
Oct 12, 2026
…nbank (#602) Hebt den Aufruf des Security-Reviews auf `c12e3c2`, den Commit von opengewerk/.github#40 auf dessen `main`. Er enthält opengewerk/.github#38 und die Korrektur aus #40. Das Review kann damit Tests ausführen, auch mit Datenbank: der Aufruf nennt die Testdatenbank dieses Repositorys, und das Review startet dafür ein eigenes PostgreSQL 18 in seiner Sandbox. Was sich am Workflow geändert hat und wie es geprüft wurde, steht in opengewerk/.github#38: Proxy der Sandbox für Node, `pnpm` im Pfad, eine Kopie des Checkouts zum Installieren, PostgreSQL in der Sandbox. Die Einstellungen der Sandbox sind unverändert. opengewerk/.github#40 verlangt dazu, dass die Schlüsseldatei der Paketquelle von PostgreSQL genau den einen erwarteten Schlüssel enthält. Teil von opengewerk/.github#37.
Musiker15
added a commit
to opengewerk/opengewerk-haustechnik
that referenced
this pull request
Oct 12, 2026
…nbank (#250) * Security-Review: Aufruf auf opengewerk/.github#38 heben, mit Testdatenbank Der Aufruf zeigt jetzt auf fb4808a, den Commit von opengewerk/.github#38 auf dessen main. Dazu nennt er die Testdatenbank dieses Repositorys. Das Review erreicht damit die Registry über den Proxy seiner Sandbox, findet pnpm im Pfad und installiert in einer Kopie des Checkouts. Vorher ließ sich in der Sandbox nichts installieren, und kein Test lief dort. * Security-Review: Aufruf auf opengewerk/.github#40 heben Der Aufruf zeigt jetzt auf c12e3c2, den Commit von opengewerk/.github#40 auf dessen main. Dort muss die Schlüsseldatei der Paketquelle von PostgreSQL genau den einen erwarteten Schlüssel enthalten; geprüft wurde vorher nur der erste der Datei.
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.
closes #39
Der Schritt "Datenbank für die Sandbox bereitstellen" prüfte vom Schlüssel der Paketquelle von PostgreSQL nur den ersten Fingerabdruck der Datei. apt vertraut aber jedem Schlüssel darin. Jetzt muss die Datei genau einen Hauptschlüssel enthalten, und zwar den erwarteten; sonst läuft das Review ohne Datenbank weiter wie bei jedem anderen Fehler dieses Schritts.
Aufgefallen im Review von opengewerk/opengewerk-haustechnik#250, das es nicht als Befund wertet.
Geprüft
Der Schritt, wie er im Workflow steht, in einem Container mit Ubuntu 24.04 gegen fünf Schlüsseldateien:
Danach
Die fünf offenen Pull Requests, die den Aufruf anheben, zeigen dann auf diesen Commit statt auf den von #38.