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

Fix for variable scope issue #392

Merged

Conversation

at0mat
Copy link

@at0mat at0mat commented Oct 14, 2022

The v2.8.6 method definition for _autoclearCookies has an issue.

The value for domains is only ever set once at the top of the method, and is overwritten by any individual row’s domain property in the cookie table blocks.

This patch ensures that any override is scoped to that particular row only.

The v2.8.6 method definition for _autoclearCookies has an issue.

The value for domains is only ever set once at the top of the method, and is overwritten by any individual row’s domain property in the cookie table blocks.

This patch ensures that any override is scoped to that particular row only.
@vercel
Copy link

vercel bot commented Oct 14, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
cookieconsent-docs ⬜️ Ignored (Inspect) Oct 14, 2022 at 7:00PM (UTC)

Copy link
Owner

@orestbida orestbida left a comment

Choose a reason for hiding this comment

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

Thanks for your contrib.!

@orestbida orestbida merged commit d201fcb into orestbida:master Oct 14, 2022
@at0mat
Copy link
Author

at0mat commented Oct 17, 2022

@orestbida not to be a pain, would it be possible to create a new patch release containing this change in the near future?

@orestbida
Copy link
Owner

@at0mat sure, expect a minor version either later today or tomorrow!

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.

None yet

2 participants