Skip to content
This repository has been archived by the owner on Apr 21, 2021. It is now read-only.

Redirect issue - endpoint choice #59

Open
sayaHub opened this issue Apr 17, 2019 · 4 comments
Open

Redirect issue - endpoint choice #59

sayaHub opened this issue Apr 17, 2019 · 4 comments

Comments

@sayaHub
Copy link
Contributor

sayaHub commented Apr 17, 2019

Use case:

  1. Service has all endpoints live (http, httpwww, https, httpswww)
  2. http, httpwww and https endpoints are configure properly and immediately redirect to the secure eventual endpoint
  3. httpswww downgrade before reaching the eventual secure endpoint

Even if we feed the scanner with a root endpoint that is secured and properly redirect to the eventual secure endpoint, tracker seems to set the canonical url to httpswww. Where there are httpswww configuration issues, it significantly impact the root domain scan result because the httpswww is chosen.

Question:
should Tracker choose to scan the secure root endpoint if Live instead of httpswww?

@obrien-j
Copy link
Contributor

Can you point to an existing domain in this situation @sayaHub ?

Point 3 there seems to imply a secure endpoint is downgrading to an insecure path.. That feels pretty "wrong".

@obrien-j obrien-j self-assigned this Apr 17, 2019
@sayaHub
Copy link
Contributor Author

sayaHub commented Apr 17, 2019

for point 3, we feel the same way, we don't understand why the configuration will be setup that way. We are trying to write up more about the right way to do redirection ...I ping you on slack about the domain example.

@obrien-j
Copy link
Contributor

image

Example of a misconfigured redirection...

The logic of how pshtt chooses the 'best' endpoint to use could be changed, but it'd probably just be easier to fix the domains showing issues.

@obrien-j obrien-j removed their assignment Apr 17, 2019
@sayaHub
Copy link
Contributor Author

sayaHub commented May 7, 2019

fyi, issue mentioned in original library :
cisagov/pshtt#89 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants