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

Delete ReformaBTL.xml #10207

Merged
merged 1 commit into from
Jun 8, 2017
Merged

Delete ReformaBTL.xml #10207

merged 1 commit into from
Jun 8, 2017

Conversation

cschanaj
Copy link
Collaborator

@cschanaj cschanaj commented Jun 7, 2017 •
edited
Loading

#9842 Only ^ support HTTPS correctly and redirects to www. However, www redirects secure connection to plaintext. Delete ReformaBTL.xml because of the limited coverage. See https://gist.github.com/cschanaj/10f7f3be8828b645a8f5ab0260c2cd1b

@cschanaj cschanaj mentioned this pull request Jun 7, 2017
Closed
@jeremyn
Copy link
Contributor

jeremyn commented Jun 8, 2017

I'm okay with deleting this, but what is https://gist.github.com/cschanaj/10f7f3be8828b645a8f5ab0260c2cd1b for?

@jeremyn jeremyn self-assigned this Jun 8, 2017
@cschanaj
Copy link
Collaborator Author

cschanaj commented Jun 8, 2017 •
edited
Loading

The ruleset for *informabtl.com on gist is meant to demonstrate the situation as described above. In case an update is preferred over a deletion, I will rebase this PR with the ruleset on gist. Sorry for the confusion incurred.

@jeremyn
Copy link
Contributor

jeremyn commented Jun 8, 2017

Okay that's fine. I'd rather delete it though than have such a limited ruleset.

@cschanaj
Copy link
Collaborator Author

cschanaj commented Jun 8, 2017

Thanks!!

@jeremyn jeremyn merged commit a0bb291 into EFForg:master Jun 8, 2017
@jeremyn jeremyn removed their assignment Jun 8, 2017
@cschanaj cschanaj deleted the cschanaj-informabtl.com branch June 8, 2017 01:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
top-100k
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cschanaj @jeremyn @Hainish