Skip to content

android: fix isExitNode check#646

Merged
kari-ts merged 1 commit into
mainfrom
kari/oop
May 20, 2025
Merged

android: fix isExitNode check#646
kari-ts merged 1 commit into
mainfrom
kari/oop

Conversation

@kari-ts

@kari-ts kari-ts commented May 8, 2025

Copy link
Copy Markdown
Collaborator

&& takes precedence over ?:, so fix isExitNode to check both IPv4 and IPv6

Updates tailscale/tailscale#15785

&& takes precedence over ?:, so fix isExitNode to check both IPv4 and IPv6

Updates tailscale/tailscale#15785

Signed-off-by: kari-ts <kari@tailscale.com>
@ghost

ghost commented May 8, 2025

Copy link
Copy Markdown

Pull Request Revisions

RevisionDescription
r1
Fixed exit node boolean logicCorrected isExitNode calculation by adding parentheses to ensure proper boolean evaluation

✅ AI review completed for r1
Help React with emojis to give feedback on AI-generated reviews:
  • 👍 means the feedback was helpful and actionable
  • 👎 means the feedback was incorrect or unhelpful
💬 Replying to feedback with a comment helps us improve the system. Your input also contributes to shaping future interactions with the AI reviewer.

We'd love to hear from you—reach out anytime at team@review.ai.

@kari-ts kari-ts requested a review from nickoneill May 8, 2025 23:55
@kari-ts kari-ts merged commit e3c76eb into main May 20, 2025
5 checks passed
@kari-ts kari-ts deleted the kari/oop branch May 20, 2025 17:42
nadeemakhter0602 pushed a commit to nadeemakhter0602/TailMon that referenced this pull request Mar 22, 2026
&& takes precedence over ?:, so fix isExitNode to check both IPv4 and IPv6

Updates tailscale/tailscale#15785

Signed-off-by: kari-ts <kari@tailscale.com>
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