caching inline templates requires too much memory

mojolicious · Sep 1, 2014 · aec221f · aec221f
1 parent ff47d9c
commit aec221f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/Changes b/Changes
@@ -1,7 +1,6 @@
 
 5.36  2014-09-02
   - Improved Mojo::Template performance.
-  - Improved performance of .ep templates.
 
 5.35  2014-08-30
   - Improved monkey_patch to be able to name generated functions.

diff --git a/lib/Mojolicious/Plugin/EPRenderer.pm b/lib/Mojolicious/Plugin/EPRenderer.pm
@@ -2,7 +2,7 @@ package Mojolicious::Plugin::EPRenderer;
 use Mojo::Base 'Mojolicious::Plugin';
 
 use Mojo::Template;
-use Mojo::Util qw(md5_sum monkey_patch);
+use Mojo::Util qw(encode md5_sum monkey_patch);
 
 sub DESTROY { Mojo::Util::_teardown(shift->{namespace}) }
 
@@ -22,7 +22,7 @@ sub register {
       my $name = $options->{inline} // $renderer->template_name($options);
       return undef unless defined $name;
       my @keys = sort grep {/^\w+$/} keys %{$c->stash};
-      my $key = join(',', $name, @keys);
+      my $key = md5_sum encode 'UTF-8', join(',', $name, @keys);
 
       # Prepare template for "epl" handler
       my $cache = $renderer->cache;