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

blocklist #741

Merged
merged 2 commits into from
Feb 5, 2015
Merged

blocklist #741

merged 2 commits into from
Feb 5, 2015

Conversation

btc
Copy link
Contributor

@btc btc commented Feb 4, 2015

This PR adds a blocklist to the gateway handler.

RFCR @whyrusleeping @jbenet

@btc btc added the status/in-progress In progress label Feb 4, 2015
}
defer f.Close()
scanner := bufio.NewScanner(f)
blocked := make(map[string]struct{}) // Implement using Bloom Filter hybrid if blocklist gets large
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good thing we have a bloom filter 😄

Brian Tiger Chow added 2 commits February 4, 2015 16:17
use pointer

use func

comment on decider to clarify whether it allows or denies

fix set conf

gstw
@btc btc modified the milestone: α Feb 5, 2015
@btc btc mentioned this pull request Feb 5, 2015
btc pushed a commit that referenced this pull request Feb 5, 2015
add blocklist to gateway executable
@btc btc merged commit 6599756 into master Feb 5, 2015
@btc btc removed the status/in-progress In progress label Feb 5, 2015
@btc btc deleted the feat/blocklist branch February 5, 2015 02:44
hacdias pushed a commit to ipfs/boxo that referenced this pull request Jan 27, 2023
add blocklist to gateway executable

This commit was moved from ipfs/kubo@6599756
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants