Skip to content

Claude/focused galileo s332u6 - #1

Merged
CodeMeAPixel merged 2 commits into
masterfrom
claude/focused-galileo-s332u6
Oct 5, 2026
Merged

CodeMeAPixel merged 2 commits into
masterfrom
claude/focused-galileo-s332u6

Conversation

@CodeMeAPixel

Copy link
Copy Markdown
Owner

No description provided.

claude added 2 commits October 5, 2026 01:21
'./install.sh' fails when the executable bit is lost (e.g. commits
from Windows); 'sh install.sh' always works. Also document that
installer variables must come after sudo, which strips them otherwise.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TXNwZBkaVRn9LobyrUJcGA
S3 destinations behind a proxy that weakens ETags (e.g. Cloudflare in
front of Alarik) failed every upload with 'ETag mismatch for part 1':
minio-go sent back W/"..." values the server could not match.

- Upload via minio Core: objects up to one part use a single PUT;
  larger ones are sent as multipart with ETags normalised (W/ stripped)
  and quoted like the AWS SDKs, plus Content-MD5 per part
- 'check' now also performs a real multipart upload
- NOBACKUPS_S3_TRACE=1 prints S3 request/response headers
- Tests model Alarik's ETag comparison behind a weakening proxy
- Docs: troubleshooting for the error and proxied endpoints

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TXNwZBkaVRn9LobyrUJcGA
@coderabbitai

coderabbitai Bot commented Oct 5, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 97beadfc-319c-4a67-a9b5-812d149c3ce8
📥 Commits

Reviewing files that changed from the base of the PR and between a9b366b and 0f6f18b.

📒 Files selected for processing (9)
  • README.md
  • cmd/nobackups/main.go
  • docs/configuration/destinations.mdx
  • docs/guides/operations.mdx
  • docs/installation.mdx
  • docs/quickstart.mdx
  • install.sh
  • internal/storage/s3.go
  • internal/storage/s3_test.go
 __________________________________________________________________
< Ah yes, the classic pattern: 'parse first, ask questions never'. >
 ------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@CodeMeAPixel
CodeMeAPixel merged commit 30282cf into master Oct 5, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants