Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

Grldk
Copy link
Contributor

@Grldk Grldk commented Mar 31, 2022

Right now Horizon does not work with a strict CSP (if unsafe-eval is not allowed). This keeps coming up every once in a while. See #1128 for the latest issue.

This PR fixes this by moving the main horizon container to a separate Vue component. I have tried this before (#1022), but my old PR failed to account for autoLoadNewEntries() in base.js, which used this.autoLoadsNewEntries, where this used to refer to the root Vue element. After these changes this refers to App.vue instead, and I failed to account for that.

This PR fixes that mistake by using this.$root in base.js and App.vue. So with this PR Horizon works as expected, and also with a CSP where unsafe-eval is not allowed.

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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.