I'm in support of this as well. As a solution i would prefer to resolve the full URL by default. Raising an exception seems to me a bit like rails telling the programmer "I know what you're intending, and we both know the solution but you have to fix it on your own."
> Here's a change in behavior I would love on the mailers. > > ## Backstory > > When you send an email, you'll likely need links. Those links must be the > full path i.e. 'http://example.com/foo' instead of relative (just '/foo'). > Unfortunately most devs are so used to using the *_path helpers, they use > them in emails by mistake. > > I made this mistake recently with http://www.codetriage.com, here's the PR > that fixes it (https://github.com/codetriage/codetriage/pull/257). The sad > part is I didn't realize there was a problem until a user got an email and > couldn't click on a link. Then they had to be nice enough to report it. If > your business is running on Rails, this may take days. I've been doing this > for years, and it still happens to me. > > > ## Feature > > Here's what I would like to see happen. Either raise an exception when *_path > helpers are used in a mail template so my tests would have caught it, or > likely just have *_path helpers resolve to the full URL by default. What do > you think? > > > -- > Richard Schneeman > > -- > You received this message because you are subscribed to the Google Groups > "Ruby on Rails: Core" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > To post to this group, send email to [email protected]. > Visit this group at http://groups.google.com/group/rubyonrails-core. > For more options, visit https://groups.google.com/d/optout. -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/rubyonrails-core. For more options, visit https://groups.google.com/d/optout.
