Remove GitHub token authentication and credential prompts from upload…

[livrasand]

…-pack handlers

Eliminado uso de GITHUB_TOKEN y headers Authorization en UploadPackDiscoveryHandler y UploadPackHandler. Removida validación de Authorization y respuesta 401 con WWW-Authenticate que forzaba negociación de credenciales. Eliminado comentario sobre prefijo anon: en ejemplo de clone/fetch en index.html.

Summary by CodeRabbit

• Bug Fixes

  • GitHub API requests no longer enforce authentication validation, removing 401 errors for missing authorization credentials.

• Chores

  • Code cleanup and minor refactoring.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This change removes GitHub token-based authentication from HTTP handlers and eliminates the associated 401 Unauthorized responses. The Authorization header setting and GITHUB_TOKEN environment variable usage have been removed from request handlers. Additionally, a comment line was removed from the HTML template file.

Changes

Cohort / File(s)	Summary
Authentication Removal internal/http/handlers.go	Removed GITHUB_TOKEN environment variable handling, os import, Authorization header setting with base64 encoding, and 401 Unauthorized error responses from UploadPackDiscoveryHandler, UploadPackHandler, and related request paths.
HTML Cleanup web/index.html	Removed single comment line from anonymous fetch terminal section.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• fix: add WWW-Authenticate header to upload-pack discovery to prevent … #34: Both PRs modify authentication behavior in internal/http/handlers.go and touch UploadPackDiscoveryHandler; this PR removes Authorization header and 401 responses while the related PR adds WWW-Authenticate header handling.

Poem

🐰 A token once guarded the GitHub gate,
But now it's removed—how light the weight!
No Authorization, no secrets to keep,
The handlers leap forward, clean and deep.
Authentication vanished with a poof,
Simpler flows, no need for proof! 🎉

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch Remove-GitHub-token-authentication-and-credential-prompts-from-upload-pack-handlers

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}