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

Delete HasGeek.xml #4085

Closed
wants to merge 1 commit into from
Closed

Delete HasGeek.xml #4085

wants to merge 1 commit into from

Conversation

karthikb351
Copy link

jobs.hasgeek.com rule no longer needed.

No longer needed.
@J0WI
Copy link
Contributor

J0WI commented Feb 6, 2016

The host had been moved to hasjob.co.
But there are still a lot of hosts on hasgeek.com.
So an updated version of the ruleset would be much more helpful than just removing.

@jeremyn
Copy link
Contributor

jeremyn commented Aug 23, 2016

@J0WI This is over six months old. Is it all right if we merge this in, and I make a new issue saying that hasgeek.com needs a new ruleset?

@J0WI
Copy link
Contributor

J0WI commented Aug 23, 2016

I would rather disable and rebase it.

@jeremyn
Copy link
Contributor

jeremyn commented Aug 23, 2016

@karthikb351 Are you interested in further updating this ruleset?

@karthikb351
Copy link
Author

We've enforced HTTPS on all our domains, so you can go ahead and merge this. There is no need for a ruleset for hasgeek.com domains.

@jeremyn
Copy link
Contributor

jeremyn commented Aug 25, 2016

HTTPS Everywhere can cover sites that redirect all HTTP to HTTPS. See here.

@J0WI
Copy link
Contributor

J0WI commented Aug 25, 2016

@karthikb351 or even better: You should consider preloaded HSTS: https://hstspreload.appspot.com/

@Hainish
Copy link
Member

Hainish commented Jan 13, 2017

Since jobs.hasgeek.com is not preloaded, this site is still vulnerable on first load to the SSLStrip attack. In order to qualify for removal, this site must be included in the HSTS preload list for all the browsers that we support. See #7126 for additional context

@Hainish Hainish closed this Jan 13, 2017
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.

4 participants