Skip to content

detect if port is used on IPV6#5508

Merged
adhami3310 merged 1 commit into
mainfrom
lendemor/fix_port_conflict_detection
Jun 27, 2025
Merged

detect if port is used on IPV6#5508
adhami3310 merged 1 commit into
mainfrom
lendemor/fix_port_conflict_detection

Conversation

@Lendemor
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Enhanced port conflict detection in reflex/utils/processes.py to support IPv6, preventing potential startup issues when ports are already in use on IPv6 addresses.

  • Updated is_process_on_port to check both IPv4 (127.0.0.1) and IPv6 (::1) localhost addresses for port conflicts
  • Critical fix for modern environments where services commonly bind to both IPv4 and IPv6 addresses

1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jun 27, 2025

CodSpeed Performance Report

Merging #5508 will not alter performance

Comparing lendemor/fix_port_conflict_detection (8638534) with main (551c5c4)

Summary

✅ 8 untouched benchmarks

@adhami3310 adhami3310 merged commit 5128f44 into main Jun 27, 2025
41 checks passed
@adhami3310 adhami3310 deleted the lendemor/fix_port_conflict_detection branch June 27, 2025 18:30
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