Skip to content

cookie: if psl fails, reject the cookie - #13033

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/psl-fail
Closed

bagder wants to merge 1 commit into
masterfrom
bagder/psl-fail

Conversation

@bagder

@bagder bagder commented Mar 3, 2024

Copy link
Copy Markdown
Member

A libpsl install without data and no built-in database is now considered bad enough to reject all cookies since they cannot be checked. It is somewhat of a user error, but still.

Reported-by: Dan Fandrich

A libpsl install without data and no built-in database is now considered
bad enough to reject all cookies since they cannot be checked. It is
somewhat of a user error, but still.

Reported-by: Dan Fandrich
@bagder bagder added the cookies label Mar 3, 2024
@github-actions github-actions Bot added the HTTP label Mar 3, 2024
@rsbeckerca

rsbeckerca commented Mar 3, 2024 via email •

Copy link
Copy Markdown
Contributor

@bagder

bagder commented Mar 3, 2024

Copy link
Copy Markdown
Member Author

Are there instructions for obtaining valid data and/or setting up a built-in database?

That's really a question for the libpsl team.

A default libpsl gets built with a built-in default PSL data set, which basically makes it impossible to fail - if there is no dynamic one to load from disk, it will fall back to the built-in set.

@rsbeckerca

Copy link
Copy Markdown
Contributor

Are there instructions for obtaining valid data and/or setting up a built-in database?

That's really a question for the libpsl team.

A default libpsl gets built with a built-in default PSL data set, which basically makes it impossible to fail - if there is no dynamic one to load from disk, it will fall back to the built-in set.

Thanks. Will go there for answers. Appreciate the reply.

@bagder bagder closed this in 9454757 Mar 4, 2024
@bagder
bagder deleted the bagder/psl-fail branch March 4, 2024 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants