Skip to content
This repository has been archived by the owner on Mar 15, 2021. It is now read-only.

Add django interpretation in email template #82

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

julienlh
Copy link
Contributor

Before accepting this pull request, it's necessary to find a way to prevent "Server Side Template Injection" .

The following test need to be passed :

  •    # test load
    
  •    content = replace_template_vars('{% load log %}{% get_admin_log 10 as admin_log %}{{ admin_log }}')
    
  •    self.assertEqual(content, '')
    

@coveralls
Copy link

coveralls commented Oct 23, 2017

Coverage Status

Coverage increased (+0.003%) to 99.657% when pulling e9ceb20 on Add_django_interpretation_in_email_template into 973b135 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 99.657% when pulling 8900a9f on Add_django_interpretation_in_email_template into 05edf0d on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 99.657% when pulling 8900a9f on Add_django_interpretation_in_email_template into 05edf0d on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 99.657% when pulling 318b592 on Add_django_interpretation_in_email_template into e73bd7f on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 99.657% when pulling 318b592 on Add_django_interpretation_in_email_template into e73bd7f on master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants