fix: #8 Begrenzte HTTP-Ausführungsengine im privaten Proxy implementieren #24

Merged

frank merged 5 commits from agent/issue-8-begrenzte-http-ausführungsengine-im-pri into main 2026-07-17 12:48:13 +02:00

Conversation 52 Commits 5 Files changed 7 +1056 -10

frank commented 2026-07-17 12:29:18 +02:00

Owner

Copy link

• Wiederverwendete HTTP-Client-/Transport-Instanz mit expliziten DNS-, Connect-, TLS-, Header- und Gesamt-Timeouts
• Harte Request-, Response-, Dekompressions- und Headerlimits
• Redirect-Limit und Zielprüfung bei jedem Hop
• Base64-kompatibles Antwortmodell mit finaler URL, Dauer und Truncation-Flag
• Typisierte Tunnel-Fehler und reguläre Behandlung von HTTP-Fehlerstatus
• Sichere Header-Normalisierung, Connection-Pooling und Idle-Close
• httptest-Abdeckung für Methoden, Redirects, Kompression, Binärdaten, 404/500, Timeout, Cancel und endlose Responses

Alle Änderungen wurden in drei Commits gepusht. go test ./... und go vet ./... sind erfolgreich; der Arbeitsbaum ist sauber.
Prüfe die HTTP-Engine gegen Scope und Akzeptanzkriterien; fehlende, klar abgegrenzte Punkte werden mit Tests ergänzt und jeweils direkt committed und gepusht.Die Engine deckt die isolierte Ausführung ab, ist aber noch nicht an den Tunnel-Request-Lifecycle gebunden. Ich ergänze die Ausführungsschicht so, dass Frame-Deadline und Cancel tatsächlich bis net/http wirken und Response-/Fehlerframes stabil erzeugt werden.Tunnel-Frames werden nun mit ihrer absoluten Deadline in einen Request-Kontext überführt; Cancel erzeugt einen terminalen Fehlerframe und schließt die laufende Upstream-Anfrage. Ein End-to-End-Test deckt diese Kette ab.Die Tunnel-Antwort wird zusätzlich auf die effektive Framegrenze begrenzt: Base64-/JSON-Overhead kann einen ansonsten zulässigen HTTP-Body nicht mehr über die 8-MiB-Framegrenze treiben.PR_TITLE: feat(http): execute bounded tunnel HTTP requests
PR_BODY: Implements and wires the bounded HTTP engine into tunnel processing, including frame deadlines, cancellation propagation, typed errors, and frame-safe Base64 response truncation. Tests pass with go test ./....

Alles Implementiert

Closes #8

- Wiederverwendete HTTP-Client-/Transport-Instanz mit expliziten DNS-, Connect-, TLS-, Header- und Gesamt-Timeouts - Harte Request-, Response-, Dekompressions- und Headerlimits - Redirect-Limit und Zielprüfung bei jedem Hop - Base64-kompatibles Antwortmodell mit finaler URL, Dauer und Truncation-Flag - Typisierte Tunnel-Fehler und reguläre Behandlung von HTTP-Fehlerstatus - Sichere Header-Normalisierung, Connection-Pooling und Idle-Close - `httptest`-Abdeckung für Methoden, Redirects, Kompression, Binärdaten, 404/500, Timeout, Cancel und endlose Responses Alle Änderungen wurden in drei Commits gepusht. `go test ./...` und `go vet ./...` sind erfolgreich; der Arbeitsbaum ist sauber. Prüfe die HTTP-Engine gegen Scope und Akzeptanzkriterien; fehlende, klar abgegrenzte Punkte werden mit Tests ergänzt und jeweils direkt committed und gepusht.Die Engine deckt die isolierte Ausführung ab, ist aber noch nicht an den Tunnel-Request-Lifecycle gebunden. Ich ergänze die Ausführungsschicht so, dass Frame-Deadline und Cancel tatsächlich bis `net/http` wirken und Response-/Fehlerframes stabil erzeugt werden.Tunnel-Frames werden nun mit ihrer absoluten Deadline in einen Request-Kontext überführt; Cancel erzeugt einen terminalen Fehlerframe und schließt die laufende Upstream-Anfrage. Ein End-to-End-Test deckt diese Kette ab.Die Tunnel-Antwort wird zusätzlich auf die effektive Framegrenze begrenzt: Base64-/JSON-Overhead kann einen ansonsten zulässigen HTTP-Body nicht mehr über die 8-MiB-Framegrenze treiben.PR_TITLE: feat(http): execute bounded tunnel HTTP requests PR_BODY: Implements and wires the bounded HTTP engine into tunnel processing, including frame deadlines, cancellation propagation, typed errors, and frame-safe Base64 response truncation. Tests pass with `go test ./...`. Alles Implementiert Closes #8

frank commented 2026-07-17 12:29:18 +02:00

Author

Owner

Copy link

🚧 WIP pull request opened — linked to issue #8. Agent is working…

🚧 **WIP pull request opened** — linked to issue #8. Agent is working…

frank commented 2026-07-17 12:29:27 +02:00

Author

Owner

Copy link

💭 Agent progress

Inspecting tool call status
Ich prüfe zunächst Repository-Struktur, bestehende Tunnel-/Policy-Typen und Tests und setze die HTTP-Engine anschließend in kleinen, separat gepushten Commits um.

💭 **Agent progress** **Inspecting tool call status** Ich prüfe zunächst Repository-Struktur, bestehende Tunnel-/Policy-Typen und Tests und setze die HTTP-Engine anschließend in kleinen, separat gepushten Commits um.

frank commented 2026-07-17 12:29:34 +02:00

Author

Owner

Copy link

💭 Agent progress

Refining command execution strategy

💭 **Agent progress** **Refining command execution strategy**

frank commented 2026-07-17 12:29:39 +02:00

Author

Owner

Copy link

💭 Agent progress

Inspecting protocol README file

💭 **Agent progress** **Inspecting protocol README file**

frank commented 2026-07-17 12:29:47 +02:00

Author

Owner

Copy link

💭 Agent progress

Designing reusable httpexec validator package

Reviewing relevant git history

💭 **Agent progress** **Designing reusable httpexec validator package** **Reviewing relevant git history**

frank commented 2026-07-17 12:31:26 +02:00

Author

Owner

Copy link

💭 Agent progress

Outlining httpexec package API

Designing request and response models with limits

Planning compressed response truncation detection

Clarifying response truncation behavior

Planning header validation and redirect handling

Initiating patch application and file creation
Die bestehende Tunnel-Spezifikation setzt 6 MiB als Obergrenze und Base64 für Bodies. Ich ergänze eine eigenständige internal/httpexec-Engine mit wiederverwendetem Transport, typisierten Protokollfehlern, Redirect-Validierung pro Hop und getrennten Limits für übertragene und dekomprimierte Antworten.

💭 **Agent progress** **Outlining httpexec package API** **Designing request and response models with limits** **Planning compressed response truncation detection** **Clarifying response truncation behavior** **Planning header validation and redirect handling** **Initiating patch application and file creation** Die bestehende Tunnel-Spezifikation setzt 6 MiB als Obergrenze und Base64 für Bodies. Ich ergänze eine eigenständige `internal/httpexec`-Engine mit wiederverwendetem Transport, typisierten Protokollfehlern, Redirect-Validierung pro Hop und getrennten Limits für übertragene und dekomprimierte Antworten.

frank commented 2026-07-17 12:31:34 +02:00

Author

Owner

Copy link

💭 Agent progress

Aligning slice index syntax

💭 **Agent progress** **Aligning slice index syntax**

frank commented 2026-07-17 12:31:42 +02:00

Author

Owner

Copy link

💭 Agent progress

Planning test compile for race diff

💭 **Agent progress** **Planning test compile for race diff**

frank commented 2026-07-17 12:32:58 +02:00

Author

Owner

Copy link

💭 Agent progress

Planning comprehensive HTTP request tests

Implementing gzip limit reader improvements

Designing HTTP header validation logic

Planning test creation and patch application

💭 **Agent progress** **Planning comprehensive HTTP request tests** **Implementing gzip limit reader improvements** **Designing HTTP header validation logic** **Planning test creation and patch application**

frank commented 2026-07-17 12:33:17 +02:00

Author

Owner

Copy link

💭 Agent progress

Running gofmt tests

Fixing sandbox cache location

💭 **Agent progress** **Running gofmt tests** **Fixing sandbox cache location**

frank commented 2026-07-17 12:33:39 +02:00

Author

Owner

Copy link

💭 Agent progress

Fixing token parsing and test assertions

💭 **Agent progress** **Fixing token parsing and test assertions**

frank commented 2026-07-17 12:34:18 +02:00

Author

Owner

Copy link

💭 Agent progress

Reviewing HTTP body size limits and truncation

Refining response body truncation with compression limits

Adding tests for endless and error responses
Die Kernfälle laufen; beim Header-Test wurde ein Map-Reihenfolgefehler sichtbar: per Connection benannte Header mussten vor dem Kopieren gesammelt werden. Das ist korrigiert. Als Nächstes sichere ich unkomprimierte/endlose Responses explizit über den Truncation-Pfad ab.

💭 **Agent progress** **Reviewing HTTP body size limits and truncation** **Refining response body truncation with compression limits** **Adding tests for endless and error responses** Die Kernfälle laufen; beim Header-Test wurde ein Map-Reihenfolgefehler sichtbar: per `Connection` benannte Header mussten vor dem Kopieren gesammelt werden. Das ist korrigiert. Als Nächstes sichere ich unkomprimierte/endlose Responses explizit über den Truncation-Pfad ab.

frank commented 2026-07-17 12:34:23 +02:00

Author

Owner

Copy link

💭 Agent progress

Running full test suite

💭 **Agent progress** **Running full test suite**

frank commented 2026-07-17 12:34:29 +02:00

Author

Owner

Copy link

💭 Agent progress

Planning diff inspection and vetting

💭 **Agent progress** **Planning diff inspection and vetting**

frank commented 2026-07-17 12:34:36 +02:00

Author

Owner

Copy link

💭 Agent progress

Staging and preparing commit
Die HTTP-Engine ist jetzt vollständig durch httptest abgedeckt, einschließlich endloser Responses; sie beendet den Upstream nach 4-KiB-Testlimit und behält nur den begrenzten Body. Ich committe und pushe diesen abgeschlossenen Kern jetzt separat.

💭 **Agent progress** **Staging and preparing commit** Die HTTP-Engine ist jetzt vollständig durch `httptest` abgedeckt, einschließlich endloser Responses; sie beendet den Upstream nach 4-KiB-Testlimit und behält nur den begrenzten Body. Ich committe und pushe diesen abgeschlossenen Kern jetzt separat.

frank added 1 commit 2026-07-17 12:34:37 +02:00

feat(http): add bounded private request engine 01f4dd05b2

frank commented 2026-07-17 12:35:46 +02:00

Author

Owner

Copy link

💭 Agent progress

Assessing engine proxy integration issues

Planning protocol enhancements and header limits

Refining response compression handling

Refining content-encoding handling

Implementing header selection with encoding control
Der erste Commit ist gepusht (01f4dd0). Im zweiten Schritt ergänze ich die noch fehlende harte Response-Header-Grenze, sichere Connection-Pool-Wiederverwendung/Shutdown per Test ab und dokumentiere die erweiterten Tunnel-Response-Felder.

Inspecting patch application status

💭 **Agent progress** **Assessing engine proxy integration issues** **Planning protocol enhancements and header limits** **Refining response compression handling** **Refining content-encoding handling** **Implementing header selection with encoding control** Der erste Commit ist gepusht (`01f4dd0`). Im zweiten Schritt ergänze ich die noch fehlende harte Response-Header-Grenze, sichere Connection-Pool-Wiederverwendung/Shutdown per Test ab und dokumentiere die erweiterten Tunnel-Response-Felder. **Inspecting patch application status**

frank commented 2026-07-17 12:35:57 +02:00

Author

Owner

Copy link

💭 Agent progress

Planning smaller patches

💭 **Agent progress** **Planning smaller patches**

frank commented 2026-07-17 12:36:19 +02:00

Author

Owner

Copy link

💭 Agent progress

Testing import locations

💭 **Agent progress** **Testing import locations**

frank commented 2026-07-17 12:36:46 +02:00

Author

Owner

Copy link

💭 Agent progress

Applying tests

💭 **Agent progress** **Applying tests**

frank commented 2026-07-17 12:36:52 +02:00

Author

Owner

Copy link

💭 Agent progress

Applying gofmt formatting

💭 **Agent progress** **Applying gofmt formatting**

frank commented 2026-07-17 12:36:59 +02:00

Author

Owner

Copy link

💭 Agent progress

Adding test for large response header

💭 **Agent progress** **Adding test for large response header**

frank commented 2026-07-17 12:37:11 +02:00

Author

Owner

Copy link

💭 Agent progress

Inspecting gofmt alignment diff

💭 **Agent progress** **Inspecting gofmt alignment diff**

frank commented 2026-07-17 12:37:22 +02:00

Author

Owner

Copy link

💭 Agent progress

Adjusting JSON indent and adding header test

💭 **Agent progress** **Adjusting JSON indent and adding header test**

frank commented 2026-07-17 12:37:29 +02:00

Author

Owner

Copy link

💭 Agent progress

Testing header size limits

💭 **Agent progress** **Testing header size limits**

frank commented 2026-07-17 12:37:36 +02:00

Author

Owner

Copy link

💭 Agent progress

Preparing commit and push
Pooling, Idle-Close und die 64-KiB-Response-Header-Grenze sind verifiziert; 404/500 bleiben reguläre Responses. Die Tunnel-v1-Dokumentation beschreibt jetzt finale URL, Base64-Kennzeichnung, Dauer und Truncation. Ich committe und pushe dieses Hardening separat.

💭 **Agent progress** **Preparing commit and push** Pooling, Idle-Close und die 64-KiB-Response-Header-Grenze sind verifiziert; 404/500 bleiben reguläre Responses. Die Tunnel-v1-Dokumentation beschreibt jetzt finale URL, Base64-Kennzeichnung, Dauer und Truncation. Ich committe und pushe dieses Hardening separat.

frank added 1 commit 2026-07-17 12:37:38 +02:00

test(http): verify pooling and response bounds fd61b67c1c

frank commented 2026-07-17 12:37:58 +02:00

Author

Owner

Copy link

💭 Agent progress

Reviewing final audit and HTTP client behaviors

Enhancing redirect URL validation and max redirects handling

Defining MaxRedirects default and validating redirects

💭 **Agent progress** **Reviewing final audit and HTTP client behaviors** **Enhancing redirect URL validation and max redirects handling** **Defining MaxRedirects default and validating redirects**

frank commented 2026-07-17 12:38:07 +02:00

Author

Owner

Copy link

💭 Agent progress

Running tests before pushing
Beim finalen Redirect-Audit fehlte noch die strukturelle URL-Prüfung auf Folgehops. Redirects mit Userinfo oder Fragment werden nun bereits vor der Policy-Prüfung als policy_denied abgewiesen.

💭 **Agent progress** **Running tests before pushing** Beim finalen Redirect-Audit fehlte noch die strukturelle URL-Prüfung auf Folgehops. Redirects mit Userinfo oder Fragment werden nun bereits vor der Policy-Prüfung als `policy_denied` abgewiesen.

frank added 1 commit 2026-07-17 12:38:09 +02:00

fix(http): validate redirect URL structure 29bfe66aca

frank commented 2026-07-17 12:38:13 +02:00

Author

Owner

Copy link

💭 Agent progress

Verifying clean commits

💭 **Agent progress** **Verifying clean commits**

frank commented 2026-07-17 12:38:21 +02:00

Author

Owner

Copy link

🔍 Verifying implementation (attempt 1/3)

🔍 **Verifying implementation** (attempt 1/3)

frank commented 2026-07-17 12:38:27 +02:00

Author

Owner

Copy link

💭 Agent progress

Composing concise German PR comment with tests

• Wiederverwendete HTTP-Client-/Transport-Instanz mit expliziten DNS-, Connect-, TLS-, Header- und Gesamt-Timeouts
• Harte Request-, Response-, Dekompressions- und Headerlimits
• Redirect-Limit und Zielprüfung bei jedem Hop
• Base64-kompatibles Antwortmodell mit finaler URL, Dauer und Truncation-Flag
• Typisierte Tunnel-Fehler und reguläre Behandlung von HTTP-Fehlerstatus
• Sichere Header-Normalisierung, Connection-Pooling und Idle-Close
• httptest-Abdeckung für Methoden, Redirects, Kompression, Binärdaten, 404/500, Timeout, Cancel und endlose Responses
  Alle Änderungen wurden in drei Commits gepusht. go test ./... und go vet ./... sind erfolgreich; der Arbeitsbaum ist sauber.Prüfe die HTTP-Engine gegen Scope und Akzeptanzkriterien; fehlende, klar abgegrenzte Punkte werden mit Tests ergänzt und jeweils direkt committed und gepusht.

💭 **Agent progress** **Composing concise German PR comment with tests** - Wiederverwendete HTTP-Client-/Transport-Instanz mit expliziten DNS-, Connect-, TLS-, Header- und Gesamt-Timeouts - Harte Request-, Response-, Dekompressions- und Headerlimits - Redirect-Limit und Zielprüfung bei jedem Hop - Base64-kompatibles Antwortmodell mit finaler URL, Dauer und Truncation-Flag - Typisierte Tunnel-Fehler und reguläre Behandlung von HTTP-Fehlerstatus - Sichere Header-Normalisierung, Connection-Pooling und Idle-Close - `httptest`-Abdeckung für Methoden, Redirects, Kompression, Binärdaten, 404/500, Timeout, Cancel und endlose Responses Alle Änderungen wurden in drei Commits gepusht. `go test ./...` und `go vet ./...` sind erfolgreich; der Arbeitsbaum ist sauber.Prüfe die HTTP-Engine gegen Scope und Akzeptanzkriterien; fehlende, klar abgegrenzte Punkte werden mit Tests ergänzt und jeweils direkt committed und gepusht.

frank commented 2026-07-17 12:38:32 +02:00

Author

Owner

Copy link

💭 Agent progress

Inspecting files and tests

💭 **Agent progress** **Inspecting files and tests**

frank commented 2026-07-17 12:38:42 +02:00

Author

Owner

Copy link

💭 Agent progress

Assessing tunnel integration and proxy use

💭 **Agent progress** **Assessing tunnel integration and proxy use**

frank commented 2026-07-17 12:38:56 +02:00

Author

Owner

Copy link

💭 Agent progress

Assessing tunnel lifecycle and connector integration

Clarifying proxy and connector design mismatch
Die Engine deckt die isolierte Ausführung ab, ist aber noch nicht an den Tunnel-Request-Lifecycle gebunden. Ich ergänze die Ausführungsschicht so, dass Frame-Deadline und Cancel tatsächlich bis net/http wirken und Response-/Fehlerframes stabil erzeugt werden.

Planning Session deadline handling

Designing connection serving with executor integration

💭 **Agent progress** **Assessing tunnel lifecycle and connector integration** **Clarifying proxy and connector design mismatch** Die Engine deckt die isolierte Ausführung ab, ist aber noch nicht an den Tunnel-Request-Lifecycle gebunden. Ich ergänze die Ausführungsschicht so, dass Frame-Deadline und Cancel tatsächlich bis `net/http` wirken und Response-/Fehlerframes stabil erzeugt werden. **Planning Session deadline handling** **Designing connection serving with executor integration**

frank commented 2026-07-17 12:39:27 +02:00

Author

Owner

Copy link

💭 Agent progress

Planning focused ExecuteProxyRequest implementation

Designing conservative tunnel connection loop

Implementing cancellation error handling

Updating deadline handling and transport loop

Refactoring listener engine injection and error handling

💭 **Agent progress** **Planning focused ExecuteProxyRequest implementation** **Designing conservative tunnel connection loop** **Implementing cancellation error handling** **Updating deadline handling and transport loop** **Refactoring listener engine injection and error handling**

frank commented 2026-07-17 12:39:51 +02:00

Author

Owner

Copy link

💭 Agent progress

Adding connector imports

💭 **Agent progress** **Adding connector imports**

frank commented 2026-07-17 12:40:00 +02:00

Author

Owner

Copy link

💭 Agent progress

Implementing session cancellation and write flushing

💭 **Agent progress** **Implementing session cancellation and write flushing**

frank commented 2026-07-17 12:40:05 +02:00

Author

Owner

Copy link

💭 Agent progress

Testing shared engine creation adjustment

💭 **Agent progress** **Testing shared engine creation adjustment**

frank commented 2026-07-17 12:40:33 +02:00

Author

Owner

Copy link

💭 Agent progress

Planning integration test using net.Pipe

Implementing cancellable HTTP test with httptest

💭 **Agent progress** **Planning integration test using net.Pipe** **Implementing cancellable HTTP test with httptest**

frank commented 2026-07-17 12:40:40 +02:00

Author

Owner

Copy link

💭 Agent progress

Planning concurrent tunnel test
Tunnel-Frames werden nun mit ihrer absoluten Deadline in einen Request-Kontext überführt; Cancel erzeugt einen terminalen Fehlerframe und schließt die laufende Upstream-Anfrage. Ein End-to-End-Test deckt diese Kette ab.

💭 **Agent progress** **Planning concurrent tunnel test** Tunnel-Frames werden nun mit ihrer absoluten Deadline in einen Request-Kontext überführt; Cancel erzeugt einen terminalen Fehlerframe und schließt die laufende Upstream-Anfrage. Ein End-to-End-Test deckt diese Kette ab.

frank commented 2026-07-17 12:41:11 +02:00

Author

Owner

Copy link

💭 Agent progress

Inspecting tunnel handler and payload limits

Adjusting response size limits and framing

Implementing response truncation with binary search

Implementing bounded response marshalling

Testing response size limits

💭 **Agent progress** **Inspecting tunnel handler and payload limits** **Adjusting response size limits and framing** **Implementing response truncation with binary search** **Implementing bounded response marshalling** **Testing response size limits**

frank commented 2026-07-17 12:41:23 +02:00

Author

Owner

Copy link

💭 Agent progress

Noting frame deadline overflow risk

Checking deadline overflow and redirect defaults
Die Tunnel-Antwort wird zusätzlich auf die effektive Framegrenze begrenzt: Base64-/JSON-Overhead kann einen ansonsten zulässigen HTTP-Body nicht mehr über die 8-MiB-Framegrenze treiben.

💭 **Agent progress** **Noting frame deadline overflow risk** **Checking deadline overflow and redirect defaults** Die Tunnel-Antwort wird zusätzlich auf die effektive Framegrenze begrenzt: Base64-/JSON-Overhead kann einen ansonsten zulässigen HTTP-Body nicht mehr über die 8-MiB-Framegrenze treiben.

frank commented 2026-07-17 12:41:29 +02:00

Author

Owner

Copy link

💭 Agent progress

Committing and pushing changes

💭 **Agent progress** **Committing and pushing changes**

frank added 1 commit 2026-07-17 12:41:30 +02:00

feat(tunnel): execute bounded HTTP requests 02ed6bccc0

frank commented 2026-07-17 12:41:50 +02:00

Author

Owner

Copy link

💭 Agent progress

Checking connection handling and closure logic

Verifying request body size handling and compression logic

Assessing response body decompression limits and updating docs

Clarifying handler state usage

💭 **Agent progress** **Checking connection handling and closure logic** **Verifying request body size handling and compression logic** **Assessing response body decompression limits and updating docs** **Clarifying handler state usage**

frank commented 2026-07-17 12:41:54 +02:00

Author

Owner

Copy link

💭 Agent progress

Inspecting unicode carriage returns

💭 **Agent progress** **Inspecting unicode carriage returns**

frank commented 2026-07-17 12:42:00 +02:00

Author

Owner

Copy link

💭 Agent progress

Applying separate line break encoding

💭 **Agent progress** **Applying separate line break encoding**

frank commented 2026-07-17 12:42:05 +02:00

Author

Owner

Copy link

💭 Agent progress

Preparing test and committing changes

💭 **Agent progress** **Preparing test and committing changes**

frank added 1 commit 2026-07-17 12:42:08 +02:00

docs(http): document frame-safe truncation e26db56db2

frank commented 2026-07-17 12:42:11 +02:00

Author

Owner

Copy link

💭 Agent progress

Inspecting git status output and cancellation behavior

💭 **Agent progress** **Inspecting git status output and cancellation behavior**

frank commented 2026-07-17 12:42:18 +02:00

Author

Owner

Copy link

✅ Verify OK — agent reported Alles Implementiert

✅ **Verify OK** — agent reported Alles Implementiert

frank changed title from WIP: #8 Begrenzte HTTP-Ausführungsengine im privaten Proxy implementieren to fix: #8 Begrenzte HTTP-Ausführungsengine im privaten Proxy implementieren 2026-07-17 12:42:19 +02:00

frank commented 2026-07-17 12:42:19 +02:00

Author

Owner

Copy link

✅ forge agent finished implementation

• Commit: docs(http): document frame-safe truncation
• Pull request: #24
• Next: waiting for CI / babysitting fixes if needed

docs/protocol/tunnel-v1.md     | 2 ++
 internal/connector/listener.go | 4 ++--
 2 files changed, 4 insertions(+), 2 deletions(-)

Agent output (tail)

Ich prüfe zunächst Repository-Struktur, bestehende Tunnel-/Policy-Typen und Tests und setze die HTTP-Engine anschließend in kleinen, separat gepushten Commits um.Die bestehende Tunnel-Spezifikation setzt 6 MiB als Obergrenze und Base64 für Bodies. Ich ergänze eine eigenständige `internal/httpexec`-Engine mit wiederverwendetem Transport, typisierten Protokollfehlern, Redirect-Validierung pro Hop und getrennten Limits für übertragene und dekomprimierte Antworten.Die Kernfälle laufen; beim Header-Test wurde ein Map-Reihenfolgefehler sichtbar: per `Connection` benannte Header mussten vor dem Kopieren gesammelt werden. Das ist korrigiert. Als Nächstes sichere ich unkomprimierte/endlose Responses explizit über den Truncation-Pfad ab.Die HTTP-Engine ist jetzt vollständig durch `httptest` abgedeckt, einschließlich endloser Responses; sie beendet den Upstream nach 4-KiB-Testlimit und behält nur den begrenzten Body. Ich committe und pushe diesen abgeschlossenen Kern jetzt separat.Der erste Commit ist gepusht (`01f4dd0`). Im zweiten Schritt ergänze ich die noch fehlende harte Response-Header-Grenze, sichere Connection-Pool-Wiederverwendung/Shutdown per Test ab und dokumentiere die erweiterten Tunnel-Response-Felder.Pooling, Idle-Close und die 64-KiB-Response-Header-Grenze sind verifiziert; 404/500 bleiben reguläre Responses. Die Tunnel-v1-Dokumentation beschreibt jetzt finale URL, Base64-Kennzeichnung, Dauer und Truncation. Ich committe und pushe dieses Hardening separat.Beim finalen Redirect-Audit fehlte noch die strukturelle URL-Prüfung auf Folgehops. Redirects mit Userinfo oder Fragment werden nun bereits vor der Policy-Prüfung als `policy_denied` abgewiesen.PR_TITLE: Begrenzte HTTP-Ausführungsengine im privaten Proxy implementieren
PR_BODY:
- Wiederverwendete HTTP-Client-/Transport-Instanz mit expliziten DNS-, Connect-, TLS-, Header- und Gesamt-Timeouts
- Harte Request-, Response-, Dekompressions- und Headerlimits
- Redirect-Limit und Zielprüfung bei jedem Hop
- Base64-kompatibles Antwortmodell mit finaler URL, Dauer und Truncation-Flag
- Typisierte Tunnel-Fehler und reguläre Behandlung von HTTP-Fehlerstatus
- Sichere Header-Normalisierung, Connection-Pooling und Idle-Close
- `httptest`-Abdeckung für Methoden, Redirects, Kompression, Binärdaten, 404/500, Timeout, Cancel und endlose Responses
Alle Änderungen wurden in drei Commits gepusht. `go test ./...` und `go vet ./...` sind erfolgreich; der Arbeitsbaum ist sauber.
Prüfe die HTTP-Engine gegen Scope und Akzeptanzkriterien; fehlende, klar abgegrenzte Punkte werden mit Tests ergänzt und jeweils direkt committed und gepusht.Die Engine deckt die isolierte Ausführung ab, ist aber noch nicht an den Tunnel-Request-Lifecycle gebunden. Ich ergänze die Ausführungsschicht so, dass Frame-Deadline und Cancel tatsächlich bis `net/http` wirken und Response-/Fehlerframes stabil erzeugt werden.Tunnel-Frames werden nun mit ihrer absoluten Deadline in einen Request-Kontext überführt; Cancel erzeugt einen terminalen Fehlerframe und schließt die laufende Upstream-Anfrage. Ein End-to-End-Test deckt diese Kette ab.Die Tunnel-Antwort wird zusätzlich auf die effektive Framegrenze begrenzt: Base64-/JSON-Overhead kann einen ansonsten zulässigen HTTP-Body nicht mehr über die 8-MiB-Framegrenze treiben.PR_TITLE: feat(http): execute bounded tunnel HTTP requests
PR_BODY: Implements and wires the bounded HTTP engine into tunnel processing, including frame deadlines, cancellation propagation, typed errors, and frame-safe Base64 response truncation. Tests pass with `go test ./...`.
Alles Implementiert

✅ **forge agent finished implementation** - Commit: `docs(http): document frame-safe truncation` - Pull request: https://repository.hildebrandt.io/ai-tools/private-proxy-mcp/pulls/24 - Next: waiting for CI / babysitting fixes if needed ``` docs/protocol/tunnel-v1.md | 2 ++ internal/connector/listener.go | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) ``` <details><summary>Agent output (tail)</summary> ``` Ich prüfe zunächst Repository-Struktur, bestehende Tunnel-/Policy-Typen und Tests und setze die HTTP-Engine anschließend in kleinen, separat gepushten Commits um.Die bestehende Tunnel-Spezifikation setzt 6 MiB als Obergrenze und Base64 für Bodies. Ich ergänze eine eigenständige `internal/httpexec`-Engine mit wiederverwendetem Transport, typisierten Protokollfehlern, Redirect-Validierung pro Hop und getrennten Limits für übertragene und dekomprimierte Antworten.Die Kernfälle laufen; beim Header-Test wurde ein Map-Reihenfolgefehler sichtbar: per `Connection` benannte Header mussten vor dem Kopieren gesammelt werden. Das ist korrigiert. Als Nächstes sichere ich unkomprimierte/endlose Responses explizit über den Truncation-Pfad ab.Die HTTP-Engine ist jetzt vollständig durch `httptest` abgedeckt, einschließlich endloser Responses; sie beendet den Upstream nach 4-KiB-Testlimit und behält nur den begrenzten Body. Ich committe und pushe diesen abgeschlossenen Kern jetzt separat.Der erste Commit ist gepusht (`01f4dd0`). Im zweiten Schritt ergänze ich die noch fehlende harte Response-Header-Grenze, sichere Connection-Pool-Wiederverwendung/Shutdown per Test ab und dokumentiere die erweiterten Tunnel-Response-Felder.Pooling, Idle-Close und die 64-KiB-Response-Header-Grenze sind verifiziert; 404/500 bleiben reguläre Responses. Die Tunnel-v1-Dokumentation beschreibt jetzt finale URL, Base64-Kennzeichnung, Dauer und Truncation. Ich committe und pushe dieses Hardening separat.Beim finalen Redirect-Audit fehlte noch die strukturelle URL-Prüfung auf Folgehops. Redirects mit Userinfo oder Fragment werden nun bereits vor der Policy-Prüfung als `policy_denied` abgewiesen.PR_TITLE: Begrenzte HTTP-Ausführungsengine im privaten Proxy implementieren PR_BODY: - Wiederverwendete HTTP-Client-/Transport-Instanz mit expliziten DNS-, Connect-, TLS-, Header- und Gesamt-Timeouts - Harte Request-, Response-, Dekompressions- und Headerlimits - Redirect-Limit und Zielprüfung bei jedem Hop - Base64-kompatibles Antwortmodell mit finaler URL, Dauer und Truncation-Flag - Typisierte Tunnel-Fehler und reguläre Behandlung von HTTP-Fehlerstatus - Sichere Header-Normalisierung, Connection-Pooling und Idle-Close - `httptest`-Abdeckung für Methoden, Redirects, Kompression, Binärdaten, 404/500, Timeout, Cancel und endlose Responses Alle Änderungen wurden in drei Commits gepusht. `go test ./...` und `go vet ./...` sind erfolgreich; der Arbeitsbaum ist sauber. Prüfe die HTTP-Engine gegen Scope und Akzeptanzkriterien; fehlende, klar abgegrenzte Punkte werden mit Tests ergänzt und jeweils direkt committed und gepusht.Die Engine deckt die isolierte Ausführung ab, ist aber noch nicht an den Tunnel-Request-Lifecycle gebunden. Ich ergänze die Ausführungsschicht so, dass Frame-Deadline und Cancel tatsächlich bis `net/http` wirken und Response-/Fehlerframes stabil erzeugt werden.Tunnel-Frames werden nun mit ihrer absoluten Deadline in einen Request-Kontext überführt; Cancel erzeugt einen terminalen Fehlerframe und schließt die laufende Upstream-Anfrage. Ein End-to-End-Test deckt diese Kette ab.Die Tunnel-Antwort wird zusätzlich auf die effektive Framegrenze begrenzt: Base64-/JSON-Overhead kann einen ansonsten zulässigen HTTP-Body nicht mehr über die 8-MiB-Framegrenze treiben.PR_TITLE: feat(http): execute bounded tunnel HTTP requests PR_BODY: Implements and wires the bounded HTTP engine into tunnel processing, including frame deadlines, cancellation propagation, typed errors, and frame-safe Base64 response truncation. Tests pass with `go test ./...`. Alles Implementiert ``` </details>

frank commented 2026-07-17 12:44:20 +02:00

Author

Owner

Copy link

⚪ No CI / build job — skipping babysit.

⚪ **No CI / build job** — skipping babysit.

frank commented 2026-07-17 12:44:21 +02:00

Author

Owner

Copy link

📊 Agent usage (codex / ACP)

In tokens	1 652 (cached read 118 272)
Out tokens	450 (thought 182)
Limits	context 54 712 / 258 400 · quota ok
Runtime	15m 2s

📊 **Agent usage** (`codex` / ACP) | | | | --- | --- | | In tokens | 1 652 (cached read 118 272) | | Out tokens | 450 (thought 182) | | Limits | context 54 712 / 258 400 · quota ok | | Runtime | 15m 2s |

frank commented 2026-07-17 12:44:21 +02:00

Author

Owner

Copy link

💭 Agent progress

Preparing final PR title and body

Ensuring exact PR body formatting

💭 **Agent progress** **Preparing final PR title and body** **Ensuring exact PR body formatting**

frank merged commit 13f5917301 into main 2026-07-17 12:48:13 +02:00

frank deleted branch agent/issue-8-begrenzte-http-ausführungsengine-im-pri 2026-07-17 12:48:14 +02:00

frank referenced this pull request from a commit 2026-07-17 12:48:14 +02:00

Merge pull request 'fix: #8 Begrenzte HTTP-Ausführungsengine im privaten Proxy implementieren' (#24) from agent/issue-8-begrenzte-http-ausführungsengine-im-pri into main

frank referenced this pull request 2026-07-17 13:01:13 +02:00

fix: #7 Private-Proxy-Modus mit Reconnect und Verbindungsübernahme implementieren #25

frank referenced this pull request 2026-07-17 13:04:40 +02:00

fix: #7 Private-Proxy-Modus mit Reconnect und Verbindungsübernahme implementieren #25

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

agent:done

Managed by forge agent

  

agent:ready

Managed by forge agent

  

agent:working

Managed by forge agent

  

area/browser

Browser-Automation und Browser-Sessions

  

area/cli

CLI und Konfiguration

  

area/http

HTTP-Request-Ausführung und Antwortverarbeitung

  

area/mcp

Model Context Protocol und MCP-Endpunkt

  

area/media

Medien-Downloads und Transkripte

  

area/observability

Logging und Health und Lifecycle

  

area/proxy

Private-Proxy-Prozess und Verbindung

  

area/security

Authentisierung und Sicherheitsrichtlinien

  

area/testing

Tests und Fuzzing und CI

  

area/tunnel

Reverse Connector und Tunnelprotokoll

  

priority/P0

MVP-kritisch oder blockiert den Kernpfad

  

priority/P1

Wichtig für ein belastbares MVP

  

priority/P2

Nachgelagertes Folgefeature

  

type/chore

Engineering und Wartungsaufgabe

  

type/design

Architektur- oder Protokollentscheidung vor Implementierung

  

type/docs

Dokumentation und Wiki

  

type/epic

Übergeordnetes Sammel- oder Lieferumfangs-Issue

  

type/feature

Neue oder erweiterte Produktfunktion

No labels

agent:done

agent:ready

agent:working

area/browser

area/cli

area/http

area/mcp

area/media

area/observability

area/proxy

area/security

area/testing

area/tunnel

priority/P0

priority/P1

priority/P2

type/chore

type/design

type/docs

type/epic

type/feature

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

ai-tools/private-proxy-mcp!24

No description provided.