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.

Sign up for GitHub

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

Jump to bottom

Remove all functions from the auto escape list that don't return a value #1547

Merged
merged 1 commit into from
Dec 16, 2018
Merged

Remove all functions from the auto escape list that don't return a value #1547

merged 1 commit into from
Dec 16, 2018

Conversation

GaryJones
Copy link
Member

Replaces #1349 by @johnbillion which has a merge conflict.

Fixes #1348.

@GaryJones GaryJones added this to the 1.x Next Release milestone Dec 16, 2018
@GaryJones GaryJones self-assigned this Dec 16, 2018
@GaryJones GaryJones mentioned this pull request Dec 16, 2018
Closed
jrfnl
jrfnl approved these changes Dec 16, 2018
View reviewed changes
Copy link
Member

@jrfnl jrfnl left a comment

Choose a reason for hiding this comment

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

As the original PR was reviewed & approved by no less than three people, let's get this in 😀

@jrfnl jrfnl merged commit f275d52 into develop Dec 16, 2018
@jrfnl jrfnl deleted the 1348-auto-escape branch December 16, 2018 20:25
@jrfnl jrfnl mentioned this pull request Dec 22, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jrfnl jrfnl jrfnl approved these changes

Assignees
No one assigned
Labels
Component: Core Focus: Security Type: Enhancement
Projects
None yet
Milestone
1.2.1
Development

Successfully merging this pull request may close these issues.

Remove all functions that don't have a return value from list of auto escaped functions
3 participants
@GaryJones @jrfnl @johnbillion