Skip to content

Conversation

@sjmiller609
Copy link
Contributor

@sjmiller609 sjmiller609 commented Oct 1, 2025

TL;DR

Clarifies the limitations and availability issues of mobile proxies in the documentation.

Why we made these changes

Mobile proxies have limited availability and do not support UDP or HTTP/2, which can lead to 502 errors and issues with some websites. This update makes these limitations clear to users to avoid confusion and support requests.

What changed?

  • proxies/mobile.mdx:
    • Significantly rewrote the 'Limitations' section to explain availability issues and the potential for 502 errors.
    • Added a recommendation to use residential proxies as a more reliable alternative.
    • Updated code examples to demonstrate carrier-based targeting instead of city-based geotargeting, which is not well-supported.

Validation

  • Verified the note renders correctly on the documentation site.

Description generated by Mesa. Update settings

Copy link
Contributor

@mesa-dot-dev mesa-dot-dev bot left a comment

Choose a reason for hiding this comment

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

Performed full review of 010a17e...01722e4

Analysis

• The PR focuses on documentation updates rather than actual architectural improvements to the mobile proxy system's reliability issues.
• Using carrier-based targeting is presented as a workaround rather than addressing the underlying availability problems in the mobile proxy system.
• The documentation now recommends residential proxies as alternatives, which may indicate the mobile proxy solution is not robust enough for many use cases.

Tip

⚡ Quick Actions

This review was generated by Mesa.

Actions:

Slash Commands:

  • /review - Request a full code review
  • /review latest - Review only changes since the last review
  • /describe - Generate PR description. This will update the PR body or issue comment depending on your configuration
  • /help - Get help with Mesa commands and configuration options

1 files reviewed | 0 comments | Review on Mesa | Edit Reviewer Settings

Copy link
Contributor

@masnwilliams masnwilliams left a comment

Choose a reason for hiding this comment

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

lgtm

@sjmiller609 sjmiller609 merged commit d478c60 into main Oct 1, 2025
2 checks passed
@sjmiller609 sjmiller609 deleted the proxy-testing branch October 1, 2025 22:50
@dprevoznik dprevoznik mentioned this pull request Jan 8, 2026
4 tasks
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.

3 participants