-
-
Notifications
You must be signed in to change notification settings - Fork 213
Redirect HTTP Traffic to HTTPS #236
Comments
@handrews is this possible? Additionally, the description on this repo should have the https link, not http. |
@epicfaace IIRC I think it's a github pages limitation? @Relequestual or @Anthropic might know? I've been focused on the spec rather than the website. |
It's not a limitation (anymore). You need to check the "Enforce HTTPS" checkbox in the Github Pages settings page. If the checkbox is not present, the site is probably using an older version of Github Pages and you may need to disable Github Pages and then re-enable; then the checkbox will show up. https://help.github.com/en/articles/securing-your-github-pages-site-with-https#enforcing-https-for-your-github-pages-site Let me know if you'd like me to help. |
@epicfaace Oh, now I remember- the problem is that the older specs specifically use I'm pretty sure this is actually a WONTFIX, since we can't selectively enforce it. |
That should be fine -- doing "force HTTPS" on Github pages simply redirects all http:// links to https:// |
@epicfaace while technically implementations should not automatically retrieve the meta-schemas from the canonical URI, if they do, it should still work and not break by suddenly returning a redirect. |
I can't see any reason to NOT do this. @handrews Older specs specifically use http as opposed to https, true, but given the redirects are seamless... I don't see it as a problem. |
I'm sure someone will complain about it but I don't care enough to argue over it. |
Redirecting with proper redirects seems fine to me. |
@philsturgeon @Relequestual as long as y'all deal with the people who complain it's fine with me. |
Yup, send them my way. |
Done! |
It looks like this got undone at some point: json-schema-org/community#39 |
@Relequestual What is the warning that GitHub shows? |
Just requires someone to investigate. If you have the time, feel free! |
Closed as completed with the new website. |
Following up on #80 would it no be possible to get http redirecting to https? Maybe even adding HSTS headers?
The text was updated successfully, but these errors were encountered: