This project is archived and is in readonly mode.
mail_to with javascript + replace_(dot|at) doesn't actually replace dots+ats
Reported by Johan Sørensen | June 1st, 2008 @ 07:07 PM
This patch makes UrlHelper#mail_to replace dots and at's when the encode option is set to javascript.
If you specified :replace_dot/:replace_at options together with the :encode => "javascript" option, the encoded javascript would actually include the raw email and not the one with dots and/or at's replaced.
Attached is a tiny git-am ready patch against 3282bf3b5016f0c9028cfff1012e8c31a13b40b7 that fixes this.
Comments and changes to this ticket
-
Repository July 12th, 2008 @ 03:01 AM
- State changed from “new” to “resolved”
- Tag set to “actionpack, bug, patch”
(from [50b5c6845ed1645cf25613024ef04187385f8dcd]) Ensure mail_to label is obfuscated for javascript encoding. [#294 state:resolved]
Signed-off-by: Pratik Naik
-
Repository August 25th, 2008 @ 09:27 AM
(from [8d61eadcc6e58a28e1c4c41c707f66a5d154d14d]) Ensure mail_to label is obfuscated for javascript encoding. [#294 state:resolved]
Signed-off-by: Pratik Naik pratiknaik@gmail.com http://github.com/rails/rails/co...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Johan Sørensen
Repository
Attachments
Tags
Referenced by
-
294
mail_to with javascript + replace_(dot|at) doesn't actually replace dots+ats
(from [8d61eadcc6e58a28e1c4c41c707f66a5d154d14d]) Ensure ...