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

Add django interpretation in email template#82

Open
julienlh wants to merge 7 commits intomasterfrom
Add_django_interpretation_in_email_template
Open

Add django interpretation in email template#82
julienlh wants to merge 7 commits intomasterfrom
Add_django_interpretation_in_email_template

Conversation

@julienlh
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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