Skip to content

Harden log-mirroring setup to use LOGS_REPO_PAT secret and remove plaintext token handling - #7

Merged
0x5t4l1n merged 7 commits into
mainfrom
copilot/create-log-automation
Apr 19, 2026
Merged

Harden log-mirroring setup to use LOGS_REPO_PAT secret and remove plaintext token handling#7
0x5t4l1n merged 7 commits into
mainfrom
copilot/create-log-automation

Conversation

Copilot AI commented Apr 19, 2026

Copy link
Copy Markdown
Contributor

The setup request relied on a raw PAT being shared directly, which is unsafe and brittle for automation. This change routes authentication through GitHub Actions secrets and makes the log-mirroring path explicit and reproducible.

  • Secret-based auth for cross-repo writes

    • Updated workflow auth to use ${{ secrets.LOGS_REPO_PAT }} for checkout/push to th30d4y/LOGS.
    • Added guardrails to fail early when the secret is missing rather than proceeding with partial execution.
  • Event logging pipeline clarity

    • Kept event mirroring append-only into daily JSONL partitions (openlearnx-events/YYYY-MM-DD/events.jsonl) to preserve auditability and simplify downstream ingestion.
  • Setup documentation

    • Documented required PAT scope and repository secret name (LOGS_REPO_PAT) so operators can complete setup without exposing credentials in code, commits, or PR text.
- uses: actions/checkout@<pinned-sha>
  with:
    repository: th30d4y/LOGS
    token: ${{ secrets.LOGS_REPO_PAT }}
    path: logs-repo

Copilot AI and others added 7 commits April 19, 2026 06:34
Agent-Logs-Url: https://github.com/th30d4y/OpenLearnX/sessions/17fba2c8-4f49-4a78-9dc4-483acc8cc945

Co-authored-by: Stalin-143 <161853795+Stalin-143@users.noreply.github.com>
Agent-Logs-Url: https://github.com/th30d4y/OpenLearnX/sessions/17fba2c8-4f49-4a78-9dc4-483acc8cc945

Co-authored-by: Stalin-143 <161853795+Stalin-143@users.noreply.github.com>
Agent-Logs-Url: https://github.com/th30d4y/OpenLearnX/sessions/17fba2c8-4f49-4a78-9dc4-483acc8cc945

Co-authored-by: Stalin-143 <161853795+Stalin-143@users.noreply.github.com>
@0x5t4l1n
0x5t4l1n marked this pull request as ready for review April 19, 2026 07:09
@0x5t4l1n
0x5t4l1n merged commit 91f5b95 into main Apr 19, 2026
2 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