Skip to content

feat: Flush after reading the archive from disk#4862

Open
another-rex wants to merge 2 commits intogoogle:masterfrom
another-rex:read-and-flush
Open

feat: Flush after reading the archive from disk#4862
another-rex wants to merge 2 commits intogoogle:masterfrom
another-rex:read-and-flush

Conversation

@another-rex
Copy link
Copy Markdown
Contributor

Attempt at advising the kernel that we don't need that file in cache anymore

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Comment thread go/cmd/gitter/gitter.go Fixed
Comment thread go/cmd/gitter/gitter.go
fileData := stdout.Bytes()

// Write to disk
f, err := os.OpenFile(archivePath, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0644)

Check failure

Code scanning / CodeQL

Uncontrolled data used in path expression High

This path depends on a
user-provided value
.
Comment thread go/cmd/gitter/gitter.go
}

fileData, err := os.ReadFile(archivePath)
f, err := os.Open(archivePath)

Check failure

Code scanning / CodeQL

Uncontrolled data used in path expression High

This path depends on a
user-provided value
.
@github-actions
Copy link
Copy Markdown

This pull request has not had any activity for 60 days and will be automatically closed in two weeks

@github-actions github-actions Bot added the stale The issue or PR is stale and pending automated closure label Apr 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale The issue or PR is stale and pending automated closure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants