Skip to content

Harden Pi-hole auth handling and Google OAuth configuration#84

Merged
aSbiEL0 merged 3 commits intomainfrom
codex/fix-api-and-oauth-issues
Mar 8, 2026
Merged

Harden Pi-hole auth handling and Google OAuth configuration#84
aSbiEL0 merged 3 commits intomainfrom
codex/fix-api-and-oauth-issues

Conversation

@aSbiEL0
Copy link
Copy Markdown
Owner

@aSbiEL0 aSbiEL0 commented Mar 8, 2026

Summary

  • require an explicit scheme for remote Pi-hole hosts and validate TLS/timeout-related configuration
  • improve Pi-hole auth handling in piholestats_v1.1.py, including v6-session vs legacy-token mode detection and clearer auth/transport failure reporting
  • tighten Google Calendar and Photos OAuth handling around loopback redirects, Desktop OAuth clients, and token scope parsing
  • add a Photos --auth-only mode and improve token-path separation from the calendar flow
  • update .env.example and README.md to document the revised Pi-hole and Google OAuth setup

Testing

  • not run from this Codex environment

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@aSbiEL0
Copy link
Copy Markdown
Owner Author

aSbiEL0 commented Mar 8, 2026

@codex[agent] please address the merge conflicts

@Codex
Copy link
Copy Markdown
Contributor

Codex AI commented Mar 8, 2026

@aSbiEL0 I've opened a new pull request, #85, to work on those changes. Once the pull request is ready, I'll request review from you.

Codex AI and others added 2 commits March 8, 2026 13:13
Co-authored-by: openai-code-agent[bot] <242516109+Codex@users.noreply.github.com>
Signed-off-by: aSbiEL0 <76663314+aSbiEL0@users.noreply.github.com>
@aSbiEL0 aSbiEL0 merged commit 4f9a864 into main Mar 8, 2026
@chatgpt-codex-connector
Copy link
Copy Markdown

Summary

Testing

  • ⚠️ curl -I -s https://git-scm.com/docs/git-merge | head -n 5 (site responded 403 in this environment)
  • rg -n "^(<<<<<<<|=======|>>>>>>>)" . || true
  • python -m py_compile scripts/calendash-api.py scripts/photos-shuffle.py scripts/piholestats_v1.1.py scripts/piholestats_v1.2.py

Committed on branch work as:

  • 316f3b1Resolve merge conflicts with latest base branch

I also created a follow-up PR via the make_pr tool with a conflict-resolution title/body on top of the original PR.

View task →

@aSbiEL0 aSbiEL0 deleted the codex/fix-api-and-oauth-issues branch March 8, 2026 13:16
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