Skip to content

Add --http-log CLI flag to replace DEBUG environment variable for HTTP request logging #77

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
pelikhan merged 7 commits into from
Jul 24, 2025
Merged

Add --http-log CLI flag to replace DEBUG environment variable for HTTP request logging #77

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
e6f4173
Initial plan
Copilot Jul 24, 2025
59fad69
Update interface and all calls to support HTTP logging filename param…
Copilot Jul 24, 2025
fca40f4
Add tests for HTTP logging filename feature and remove DEBUG env var …
Copilot Jul 24, 2025
a46680e
Refactor HTTP log to use Context instead of function parameters
Copilot Jul 24, 2025
3cdc67e
Group HTTP log writes into single fprintf call for better performance
Copilot Jul 24, 2025
d19f533
Remove test files as requested
Copilot Jul 24, 2025
7903afa
Update internal/azuremodels/azure_client.go
pelikhan Jul 24, 2025
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
Prev Previous commit
Update internal/azuremodels/azure_client.go 
Co-authored-by: Copilot <[email protected]>
  • Loading branch information
@pelikhan
pelikhan and Copilot authored Jul 24, 2025
commit 7903afab145fe95b664e37b5d5f76d7c35c0aa23
3 changes: 2 additions & 1 deletion internal/azuremodels/azure_client.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,9 @@
if err == nil {
defer logFile.Close()
fmt.Fprintf(logFile, "### %s\nPOST %s\nAuthorization: Bearer {{$processEnv GITHUB_TOKEN}}\nContent-Type: application/json\nx-ms-useragent: github-cli-models\nx-ms-user-agent: github-cli-models\n\n%s\n\n",
time.Now().Format(time.RFC3339), inferenceURL, string(bodyBytes))
const logFormat = "### %s\nPOST %s\nAuthorization: Bearer {{$processEnv GITHUB_TOKEN}}\nContent-Type: application/json\nx-ms-useragent: github-cli-models\nx-ms-user-agent: github-cli-models\n\n%s\n\n"

Check failure on line 77 in internal/azuremodels/azure_client.go

View workflow job for this annotation

GitHub Actions / lint 

expected operand, found 'const' (typecheck)

Check failure on line 77 in internal/azuremodels/azure_client.go

View workflow job for this annotation

GitHub Actions / lint 

syntax error: unexpected const, expected expression (typecheck)

Check failure on line 77 in internal/azuremodels/azure_client.go

View workflow job for this annotation

GitHub Actions / lint 

syntax error: unexpected const, expected expression) (typecheck)

Check failure on line 77 in internal/azuremodels/azure_client.go

View workflow job for this annotation

GitHub Actions / lint 

syntax error: unexpected const, expected expression) (typecheck)

Check failure on line 77 in internal/azuremodels/azure_client.go

View workflow job for this annotation

GitHub Actions / lint 

syntax error: unexpected const, expected expression) (typecheck)

Check failure on line 77 in internal/azuremodels/azure_client.go

View workflow job for this annotation

GitHub Actions / build 

syntax error: unexpected keyword const, expected expression
fmt.Fprintf(logFile, logFormat, time.Now().Format(time.RFC3339), inferenceURL, string(bodyBytes))

Check failure on line 78 in internal/azuremodels/azure_client.go

View workflow job for this annotation

GitHub Actions / lint 

missing ',' before newline in argument list (typecheck)
}

Check failure on line 79 in internal/azuremodels/azure_client.go

View workflow job for this annotation

GitHub Actions / lint 

expected operand, found '}' (typecheck)
}

httpReq, err := http.NewRequestWithContext(ctx, http.MethodPost, inferenceURL, body)
Expand Down
Loading