Skip to content

Fix: Update-Token sicher in _token.py auslagern#4

Merged
oliverba81 merged 2 commits into
mainfrom
feature/session-updates
May 29, 2026
Merged

Fix: Update-Token sicher in _token.py auslagern#4
oliverba81 merged 2 commits into
mainfrom
feature/session-updates

Conversation

@oliverba81

Copy link
Copy Markdown
Owner

Summary

  • GitHub revoziert PATs automatisch wenn sie in Commits landen → Token aus dem Code entfernt
  • Update-Token wird jetzt aus _token.py importiert (liegt neben der .py-Datei, nie im Repo)
  • _token.py in .gitignore aufgenommen
  • GitHub-Token-Feld aus Einstellungen entfernt (kein manueller Eintrag mehr nötig)

Token einrichten

  1. Neuen PAT erstellen: https://github.com/settings/personal-access-tokens/new
    → Repository oliverba81/website-scraper → Contents: Read-only
  2. Token in _token.py eintragen: GITHUB_UPDATE_TOKEN = "github_pat_..."
  3. Beim Verteilen: website_scraper.py + _token.py zusammen weitergeben

Test plan

  • _token.py mit gültigem Token anlegen → App starten → Update-Dialog nach 3 Sek.
  • _token.py fehlt → App startet ohne Fehler, kein Update-Dialog
  • Einstellungen → kein GitHub-Token-Feld mehr sichtbar

🤖 Generated with Claude Code

oliverba81 and others added 2 commits May 29, 2026 14:15
All distributed users now get update notifications without any
configuration. Custom token in settings still takes priority.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
GitHub auto-revokes PATs committed to repos. Token now lives in
_token.py (gitignored) next to the .py file - safe to distribute,
never enters the repo. Removed GitHub token field from Settings dialog.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@oliverba81 oliverba81 merged commit af631d3 into main May 29, 2026
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.

1 participant