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

Refactor to separate uses of IP #38

Merged
merged 2 commits into from
Dec 12, 2023
Merged

Refactor to separate uses of IP #38

merged 2 commits into from
Dec 12, 2023

Conversation

DavidSchinazi
Copy link
Collaborator

@DavidSchinazi DavidSchinazi commented Oct 6, 2023

As proposed in #36, this PR attempts to differentiate uses of IP into two broad categories: personalization and anti-abuse. It then refactors some sections to leverage that separation. Some further tweaks will be needed after but I think this improves how the document reads.

Here is a rendered view of this PR.

Fixes #36

@bakkot
Copy link

bakkot commented Oct 6, 2023

I don't think the geolocation use case really makes sense as either tracking or anti-abuse.

determining "reputation" {{!RFC5782}} in conjunction with other signals to
Tracking of IP addresses is common place on the Internet today, and falls
roughly into two broad categories. The first is profiling, the creation of
interest profiles for a given user to serve them relevant advertisements. The
Copy link
Collaborator

Choose a reason for hiding this comment

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

If we define profiling just in terms of advertising then I tend to agree with @bakkot that geolocation doesn't fit well in either bucket. May I suggest that the category be personalization, of which profiling for ads is one example of personalization. Other examples would be tailoring content to your location (such as local news, or local stores).

@DavidSchinazi
Copy link
Collaborator Author

Good point, I've replaced profiling with personalization. Let me know what you think

([ODoH]({{?I-D.pauly-dprive-oblivious-doh}})) and Oblivious HTTP
([OHTTP]({{?I-D.thomson-ohai-ohttp}})). While they both prevent tracking by
individual parties, they are not intended for the general-purpose web
browsing use case.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should probably expand on this to explain why they're not a general purpose solution. This can be a follow up bug though.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah that text predates this refactor so let's address that separately.

@DavidSchinazi DavidSchinazi merged commit c88801d into main Dec 12, 2023
2 checks passed
@DavidSchinazi DavidSchinazi deleted the refactor36 branch December 12, 2023 23:00
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.

Potential tweak to structure of document
3 participants