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

Disallow calling "helperMissing" and "blockHelperMissing" directly #1559

Merged
merged 2 commits into from
Sep 24, 2019

Conversation

nknapp
Copy link
Collaborator

@nknapp nknapp commented Sep 21, 2019

closes #1558

@nknapp nknapp force-pushed the forbid-calling-missing-helper branch from aad5ad8 to 7b67a29 Compare September 21, 2019 13:54
@nknapp nknapp force-pushed the forbid-calling-missing-helper branch from f6a6b71 to c12c5a9 Compare September 21, 2019 14:04
@nknapp
Copy link
Collaborator Author

nknapp commented Sep 21, 2019

This PR essentially moves the helperMissing and blockHelperMissing to a hooks-object in the container. The compiler uses the the hooks object to call them when a helper is missing.

That way, the helpers cannot be called directly anymore.

@nknapp
Copy link
Collaborator Author

nknapp commented Sep 24, 2019

I'll release this change now. Because I want to talk about the general issue at a user-group and I feel better if the current security issue is not working like this anymore.

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.

1 participant