Skip to content
This repository has been archived by the owner on Jan 18, 2018. It is now read-only.

Remove sites not affected by this vuln #28

Closed
coderobe opened this issue Feb 24, 2017 · 5 comments
Closed

Remove sites not affected by this vuln #28

coderobe opened this issue Feb 24, 2017 · 5 comments

Comments

@coderobe
Copy link
Contributor

Several (currently listed) sites, including coderobe.net and rob1nn.pw which i'm the provable owner of were not affected by this vulnerability. This should:

  • Be stated in readme.md
    OR:
  • Removed from the zip entirely

Now that this repository is making the news, this could potentially hurt the reputation of many sites.

@coderobe
Copy link
Contributor Author

Creating PRs for this is not very useful, because the file is zipped - which makes merging multiple PRs nearly impossible / an infeasible thing to do. I'd suggest a switch to keeping the file(s) uncompressed, potentially split into multiple files.

@pirate
Copy link
Owner

pirate commented Feb 24, 2017

@coderobe good ideas on both count. I'll be updating the README shortly and uploading the unzipped file.

@pirate
Copy link
Owner

pirate commented Feb 24, 2017

I've added a disclaimer to the README: https://github.com/pirate/sites-using-cloudflare#full-list
The text file has been uploaded. Submit a PR to remove your items, (I can recompress it for you, just PR the text file).

@pirate
Copy link
Owner

pirate commented Feb 24, 2017

@coderobe happy to remove your sites as they're personal and don't appear to have any user data. Just submit a PR and I'll recompress the zip for you.

@pirate pirate closed this as completed Feb 24, 2017
@coderobe
Copy link
Contributor Author

Here's the PR btw #117

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