Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: automate removal of domains called out as dangerous #395

Open
SgtPooki opened this issue Mar 24, 2023 · 4 comments
Open

feat: automate removal of domains called out as dangerous #395

SgtPooki opened this issue Mar 24, 2023 · 4 comments
Labels
kind/enhancement A net-new feature or improvement to an existing feature need/analysis Needs further analysis before proceeding need/author-input Needs input from the original author

Comments

@SgtPooki
Copy link
Member

SgtPooki commented Mar 24, 2023

since we do no security audits ourselves, google, bing, cloudflare, etc.. may have much more information about errors like the one called out initially by #392 and resolved by #394.

We should be able to detect this, and remove the URLs, via some CI or automated action.

Related issues:

@SgtPooki SgtPooki added the need/triage Needs initial labeling and prioritization label Mar 24, 2023
@whizzzkid
Copy link
Contributor

There's some work in progress regarding this:

This issue has been discussed here: https://blog.nft.storage/posts/2022-04-29-gateways-and-gatekeepers

Overall I believe this could become a tiring game of whack-a-mole, there always will be malicious actors. It might be worth exploring a public, community-maintained list of such deceptive content which could alleviate some concern. This may not scale in the longer term so would need a more robust solution.

@github-project-automation github-project-automation bot moved this to Needs Grooming in IPFS-GUI (PL EngRes) Mar 25, 2023
@whizzzkid whizzzkid added kind/enhancement A net-new feature or improvement to an existing feature need/analysis Needs further analysis before proceeding need/author-input Needs input from the original author and removed need/triage Needs initial labeling and prioritization labels Mar 30, 2023
@meandavejustice
Copy link

image

via0.com is giving me warning today in brave

@whizzzkid
Copy link
Contributor

@meandavejustice weird via0.com is unreachable for me. 🤔

@meandavejustice
Copy link

Not seeing the warning anymore 🤷

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A net-new feature or improvement to an existing feature need/analysis Needs further analysis before proceeding need/author-input Needs input from the original author
Projects
No open projects
Status: Needs Grooming
Development

No branches or pull requests

3 participants