Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Update and combine Steam related rulesets #15087

Merged
merged 1 commit into from
Apr 6, 2018
Merged

Update and combine Steam related rulesets #15087

merged 1 commit into from
Apr 6, 2018

Conversation

xPaw
Copy link
Contributor

@xPaw xPaw commented Apr 6, 2018

Fixes #15086.

Valve just recently stopped redirecting https to http, as they're preparing for full https roll out. Steam community already forces https.

@Bisaloo
Copy link
Collaborator

Bisaloo commented Apr 6, 2018

@xPaw, why did you put ^https? on all rules?

We are trying to get rid of HTTPS->HTTPS rewrites (#10843).

@xPaw
Copy link
Contributor Author

xPaw commented Apr 6, 2018

Because these URLs don't serve correct certificate or content when requested over https. Technically nothing would break if you changed it to ^http as these subdomains are not referenced anywhere directly.

@Bisaloo
Copy link
Collaborator

Bisaloo commented Apr 6, 2018

Okay. It would be useful if you could open a follow up PR to fix this.

@dxgldotorg
Copy link
Contributor

Maybe now add a ruleset for store.steampowered.com to circumvent the requirement that users' date of birth be submitted via plaintext http?

@xPaw
Copy link
Contributor Author

xPaw commented Apr 6, 2018

@DXGLdotinfo Does https everywhere rewrite form submissions? Because I have created a rule to redirect the entire subdomain to https.

I wouldn't rush adding other rules because Valve will be further improving their https setup.

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

Successfully merging this pull request may close these issues.

5 participants