[Yanel-dev] Pull Request: Support EXPIRES and IF-MODIFIED-SINCE

[Michael Wechner]

Hi Balz

I have refactored your pull request:

- made Expires configurable, for example

<yanel:resource-config xmlns:yanel="http://www.wyona.org/yanel/rti/1.0">
  <yanel:rti name="file"
namespace="http://www.wyona.org/yanel/resource/1.0"/>

  <yanel:property name="days-to-expiration" value="7"/>
</yanel:resource-config>


- I have removed your code re last modified, because this resource
implements ModifiableV2, such that YanelServlet will check on the last
modified and it does not need to be implemented by the resource itself.

Please give it a try and thanks again for your patch

Thanks

Michael


Am 26.05.16 um 10:20 schrieb basZero:
>
> Hi,
>
> I submitted another pull request:
>
> https://github.com/wyona/yanel/pull/80/files
>
> For a framework like Yanel I think it is crucial to fully support
> todays recommendations stated by Google as most probably all web
> applications in the internet are heavily depending on the SEO ranking
> at Google.
>
> So this pull request implements the requirements for browser caching:
>
>   * Set |Expires| correctly
>   * Support |If-Modified-Since|
>
> This pull request includes two private methods. These are candidates
> to be moved into a central helper class, maybe as static methods to be
> used anywhere in Yanel.
>
>
> Cheers, Bas
>
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mx2.wyona.com/pipermail/yanel-development/attachments/20160619/f1bee3d4/attachment.html>