Skip to content

Add custom port configuration support#149

Merged
levz0r merged 6 commits into
masterfrom
patch-1
Dec 4, 2025
Merged

Add custom port configuration support#149
levz0r merged 6 commits into
masterfrom
patch-1

Conversation

@levz0r
Copy link
Copy Markdown
Owner

@levz0r levz0r commented Dec 4, 2025

Summary

  • Add optional port parameter to all public API functions (check_inbox, get_messages, refresh_access_token)
  • Fix bug where port parameter was being reassigned instead of passed through (3 locations)
  • Change default port from 80 to 32019 (port 80 requires root/admin privileges)
  • Update init.js to accept optional custom port argument
  • Add redirect URI documentation to README

Test plan

  • Added unit tests for port parameter passing
  • All 9 tests pass

Supersedes #135

🤖 Generated with Claude Code

@levz0r levz0r mentioned this pull request Dec 4, 2025
Revadike and others added 6 commits December 4, 2025 18:19
- Fix port parameter being reassigned instead of passed through in 3 places
- Change default port from 80 to 32019 (port 80 requires root/admin privileges)
- Add redirect URI documentation to README
- Update init.js to support optional custom port argument
- Add tests for port parameter passing
- Change oauth2 import style in gmail.js for testability

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@levz0r levz0r merged commit 0748e01 into master Dec 4, 2025
2 checks passed
@levz0r levz0r deleted the patch-1 branch December 4, 2025 18:13
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