bramus / bramus/mixed-content-scan

When following redirects for the rootUrl, the rootUrl can get scanned twice

Open
#5 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
PHP
Stars
517
Forks
50
PR merge metrics
No merged PRs in 30d

Description

```
$ php bin/scanner.php https://www.wijs.be/en
Scanning https://www.wijs.be/en/
[2014-12-10 18:18:32] 00000 - https://www.wijs.be/en/
> Updated rootUrl to https://wijs.be/en/
[2014-12-10 18:18:32] 00001 - https://wijs.be/en/internships
[2014-12-10 18:18:32] 00002 - https://wijs.be/en/jobs
[2014-12-10 18:18:32] 00003 - https://wijs.be/en/
> Updated rootUrl to https://wijs.be/en/
[2014-12-10 18:18:32] 00004 - https://wijs.be/en/services
[2014-12-10 18:18:33] 00005 - https://wijs.be/en/about-wijs
[2014-12-10 18:18:33] 00006 - https://wijs.be/en/contact
[2014-12-10 18:18:33] 00007 - https://wijs.be/en/services/strategy-concept
[2014-12-10 18:18:33] 00008 - https://wijs.be/en/services/designing-building
[2014-12-10 18:18:33] 00009 - https://wijs.be/en/services/generating-traffic
[2014-12-10 18:18:34] 00010 - https://wijs.be/en/portfolio/detail/world-scout-education-congress
- http://www.facebook.com/plugins/like.php?href=https://wijs.be/en/portfolio/detail/world-scout-education-congress&send=false&layout=button_count&show_faces=false&colorscheme=&font=&height=21
- http://platform.linkedin.com/in.js
[2014-12-10 18:18:34] 00011 - https://wijs.be/en/services/strategy-concept/context-study

...
```

This is most likely triggered by the forcing of the trailing `/` on URLs. Needs further investigation.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by running php bin/scanner.php with a redirecting root URL such as https://www.wijs.be/en and trace how the rootUrl and trailing slash are handled. The work is complete when the redirected root URL is scanned only once while the remaining links are still discovered.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.