Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MatchPatternUrl and UrlRegex are not working correctly #311

Closed
sik0vny opened this issue Aug 1, 2023 · 1 comment
Closed

MatchPatternUrl and UrlRegex are not working correctly #311

sik0vny opened this issue Aug 1, 2023 · 1 comment
Labels
type-bug Bug in code or behaviour

Comments

@sik0vny
Copy link

sik0vny commented Aug 1, 2023

Description

I want to catch all URLs with "MapServer" string in them.

Here is an example:
https://ags.cuzk.cz/arcgis1/rest/services/ZTM/MapServer/tile/4/2126/2012?blankTile=false

Both MatchPatternUrl and RegexUrl are not catching it.
I tried:

  • MatchPatternUrl: *://*/*MapServer*
  • MatchPatternUrl: *://*MapServer*
  • RegexUrl: .*MapServer.*

Or am I doing something wrong?

To be sure I have everything else configured correctly, I tried: MatchPatternUrl: *://*/* , and it catches all URLs correctly ( I can see them logged in my Proxy server)

Steps to Reproduce

Set SmartProxy option with any of the above examples
Start your proxy server

Go to https://ags.cuzk.cz/geoprohlizec/
Start zooming into the map
Many urls https://ags.cuzk.cz/arcgis1/rest/services/ZTM/MapServer/tile* will be requested

None of them is being redirected to the proxy

Which browsers did you test this on?

Firefox 114.0.2 (64-bit)

Affected browser versions

No response

Affected SmartProxy versions

1.3.0

Screenshots of the problems or steps to reproduce

No response

Any additional context

No response

@sik0vny sik0vny added the type-bug Bug in code or behaviour label Aug 1, 2023
@salarcode
Copy link
Owner

Sorry for the late reply, I tried this just now.
There is a bug in using lowercase urls for regex and patterns that causes this. It will be fixed with next version.

For now you can use the lowercase of your rules:

MatchPatternUrl: *://*/*mapserver*
RegexUrl: .*mapserver.*

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug Bug in code or behaviour
Projects
None yet
Development

No branches or pull requests

2 participants