Skip to content

Commit e8a363b

Browse files
Merge pull request #38 from blackbullion/injection-fix
Fix injected scripts not executing
2 parents 443df27 + 02efc04 commit e8a363b

File tree

3 files changed

+11
-3
lines changed

3 files changed

+11
-3
lines changed

dist/cookies.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/script.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

resources/js/Cookies.js

+9-1
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,18 @@ class LaravelCookieConsent {
3737
}
3838

3939
data.scripts.forEach(script => {
40+
const scriptRegex = /<script.*<\/script>/;
41+
if (!scriptRegex.test(script)) {
42+
console.error('Invalid script tag: ' + script);
43+
}
4044
let tmp = document.createElement('div');
4145
tmp.innerHTML = script;
4246

43-
let tag = tmp.querySelector('script')
47+
let tag = document.createElement('script');
48+
tag.textContent = tmp.querySelector('script').textContent;
49+
for (const attr of tmp.querySelector('script').attributes) {
50+
tag.setAttribute(attr.name, attr.value);
51+
}
4452
tag.setAttribute('data-cookie-consent', true);
4553

4654
document.head.appendChild(tag);

0 commit comments

Comments
 (0)