Definitely. 

On Jul 9, 2013, at 6:26 PM, Michael Koziarski <notifications@github.com> wrote:

> In activesupport/lib/active_support/message_verifier.rb:
> 
> > +      parts = signed_message.split("--")
> > +      if parts.length == 3
> > +        data, expires, digest = parts
> > +      else
> > +        data, digest = parts
> > +        expires = nil
> > +      end
> > +
> > +      if data.present? && digest.present? && secure_compare(digest, generate_digest(data, expires))
> > +        value = @serializer.load(::Base64.decode64(data))
> > +
> > +        if expires
> > +          if Time.at(Integer(expires)) >= Time.now
> > +            value
> > +          else
> > +            raise Expired
> Would be nice to include the time that it expired in this message, obviously we don't include the failed digest but in this case I can't see a reason not to include it
> 
> —
> Reply to this email directly or view it on GitHub.