Skip to content

Fix Content-Security-Policy violation#263

Open
vtenfys wants to merge 1 commit into
eyalroz:masterfrom
vtenfys:fix-csp
Open

Fix Content-Security-Policy violation#263
vtenfys wants to merge 1 commit into
eyalroz:masterfrom
vtenfys:fix-csp

Conversation

@vtenfys

@vtenfys vtenfys commented Apr 6, 2025

Copy link
Copy Markdown

Fixes #262

@eyalroz

eyalroz commented Apr 6, 2025

Copy link
Copy Markdown
Owner

Can this really fix anything? it does the exact same thing, doesn't it? 8-(

@vtenfys

vtenfys commented Apr 6, 2025

Copy link
Copy Markdown
Author

It surprised me too, but it does fix my issue!

ChatGPT offers the following explanation:

image

I think the key issue is the missing function binding.

I haven't tested it, but I expect the following version would also work:

setTimeout(this.waitForFolderCollection.bind(this, searchData), 100);

@eyalroz

eyalroz commented Apr 10, 2025

Copy link
Copy Markdown
Owner

So, I'll accept this, but - not against the master branch, it needs to happen against the development branch. The master branch always has commits up to the latest release.

Thanks.

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.

Thunderbird (sometimes) fails to accept eval-ish setTimeOut calls

2 participants