Skip to content

Conversation

@prathik2401
Copy link

@prathik2401 prathik2401 commented Sep 8, 2025

Bug 2491, which stated that the URLs were being converted to lowercase, which potentially caused issues for URLs with uppercase characters.

Describe the Pull Request

All the URLs in Static Analysis were being forced to lowercase characters, which fails in real-world, because URLs are case sensitive, and only domains are case insensitive.

Checklist for PR

  • Run MobSF unit tests and lint tox -e lint,test
  • Tested Working on Linux, Mac, Windows, and Docker
  • Add unit test for any new Web API (Refer: StaticAnalyzer/tests.py)
  • Make sure tests are passing on your PR MobSF tests

Additional Comments (if any)

I have tested this on couple of applications which have lowercase, uppercase, and mixture of both these. And now we are able to capture the case of the url too.

prathik-24 and others added 2 commits September 8, 2025 17:16
Bug 2491, which stated that the URLs were being converted to lowercase, which potentially caused issues for URLs with uppercase characters.
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