Commit 33554e0
fix: proxy response envelope preservation for GraphQL and search
Three bugs fixed:
1. copyResponseHeaders now copies Content-Type from upstream, fixing
GraphQL passthrough where octokit couldn't parse the response
2. Search routes now pass the 'q' query param as 'query' arg to the
guard, enabling proper repo-scoped integrity labels
3. REST search responses are re-wrapped in the original envelope
({total_count, items}) after filtering. GraphQL responses with
filtered items are reconstructed preserving the {data:{...}} tree
Smoke workflow updated to build from source (docker build) instead
of pulling a published image, so we can test unreleased fixes.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>1 parent 8531f41 commit 33554e0
File tree
4 files changed
+231
-20
lines changed- .github/workflows
- internal/proxy
4 files changed
+231
-20
lines changedSome generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
| 26 | + | |
26 | 27 | | |
27 | 28 | | |
28 | 29 | | |
| |||
44 | 45 | | |
45 | 46 | | |
46 | 47 | | |
47 | | - | |
48 | | - | |
49 | | - | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
50 | 65 | | |
51 | 66 | | |
52 | 67 | | |
| |||
66 | 81 | | |
67 | 82 | | |
68 | 83 | | |
69 | | - | |
| 84 | + | |
70 | 85 | | |
71 | 86 | | |
72 | 87 | | |
| |||
393 | 408 | | |
394 | 409 | | |
395 | 410 | | |
396 | | - | |
| 411 | + | |
397 | 412 | | |
398 | 413 | | |
399 | 414 | | |
| |||
0 commit comments