<div dir="ltr">Hi Michael, <div>no this configuration approach you have chosen is ok for me, I just didn't understand you initially, now I see the point.</div><div><br></div><div>Thanks for that. Cheers, Balz</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 8, 2016 at 9:38 PM, Michael Wechner <span dir="ltr"><<a href="mailto:michael.wechner@wyona.com" target="_blank">michael.wechner@wyona.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
<div>Hi Balz<br>
<br>
See my comments below<br>
<br>
Am 08.06.16 um 15:11 schrieb basZero:<br>
</div><span class="">
<blockquote type="cite">
<div dir="ltr">Hi Michael,
<div><br>
</div>
<div>I don't really understand your statement "then the browser
does not check anymore whether the location has changed", can
you give me more info on that? URL?</div>
</div>
</blockquote>
<br></span>
when you access for example on one day<br>
<br>
<a href="http://foo.bar" target="_blank">http://foo.bar</a><br>
<br>
and on the next day you access it again<br>
<br>
<a href="http://foo.bar" target="_blank">http://foo.bar</a><br>
<br>
When you set on this URL a permanent redirect, then for example
Firefox does not check again the location, but uses the cached
redirect of the day before.<span class=""><br>
<br>
<blockquote type="cite">
<div dir="ltr">
<div><br>
</div>
<div>My pull request was to make redirects better for search
engines (SEO). So what is done for logged in users is anyway
irrelevant for SEO, as all search engines are anonymous
visitors.</div>
</div>
</blockquote>
<br></span>
yes, but for dynamic redirects the difference between 301 and 307 is
not irrelavent, but crucial.<br>
<br>
I do understand the importance of your requirement, but the
requirement of temporary/dynamic redirects is also important, and
because so far we used 307 as default, we cannot just change to 301
as default, because the expected behaviour would not work anymore.<br>
<br>
As said, the alternative would be introduce a new version of the
redirect resource and make 301 the default. Then everyone will be
happy :-) I can do this if you think configuring 301 inside the
resource configuration is too complicated. <br>
<br>
Thanks<span class="HOEnZb"><font color="#888888"><br>
<br>
Michael</font></span><div><div class="h5"><br>
<blockquote type="cite">
<div dir="ltr">
<div><br>
</div>
<div>Thanks</div>
</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Wed, Jun 8, 2016 at 2:59 PM, Michael
Wechner <span dir="ltr"><<a href="mailto:michael.wechner@wyona.com" target="_blank">michael.wechner@wyona.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
<div>Hi Balz<br>
<br>
I understand your argument about the "link juice", but
there are situations like for example the following:<br>
<br>
<br>
Not-signed in user:<br>
<br>
<a href="http://foo.bar" target="_blank">http://foo.bar</a> -> <a href="http://foo.bar/en/index.html" target="_blank"></a><a href="http://foo.bar/en/index.html" target="_blank">http://foo.bar/en/index.html</a><br>
<br>
Signed in user:<br>
<br>
<a href="http://foo.bar" target="_blank">http://foo.bar</a> -> <a href="http://foo.bar/en/projects.html" target="_blank"></a><a href="http://foo.bar/en/projects.html" target="_blank">http://foo.bar/en/projects.html</a><br>
<br>
Now the second case might change at some point or is
even dynamic, e.g.<br>
<br>
<a href="http://foo.bar" target="_blank">http://foo.bar</a> -> <a href="http://foo.bar/en/overview.html" target="_blank"></a><a href="http://foo.bar/en/overview.html" target="_blank">http://foo.bar/en/overview.html</a><br>
<br>
When one is using a 301, then the browser does not check
anymore whether the location has changed and such a case
will fail. Using 307 it will work.<br>
<br>
But let me see what I can do :-)<br>
<br>
Thanks<br>
<br>
Michael<br>
<br>
Am 08.06.16 um 13:39 schrieb basZero:<br>
</div>
<div>
<div>
<blockquote type="cite">
<div dir="ltr">Yes,
<div>but to be honest, I would set 301 as default
but I understand that you want to keep Yanel as
it was (backwards compatibility).</div>
<div><br>
</div>
<div>However I think that having 301 by default in
place would help all web applications. Nobody
want to avoid the "link juice" benefit.</div>
<div><br>
</div>
<div>Cheers, Balz</div>
</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Wed, Jun 8, 2016 at
1:31 PM, Michael Wechner <span dir="ltr"><<a href="mailto:michael.wechner@wyona.com" target="_blank"></a><a href="mailto:michael.wechner@wyona.com" target="_blank">michael.wechner@wyona.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
<div>Hi Balz<br>
<br>
You are right. I will do better with the
next pull request.<br>
<br>
But what about the functionality itself,
is it good for you as I have done it?<br>
<br>
Thanks<br>
<br>
Michael<br>
<br>
Am 08.06.16 um 11:00 schrieb basZero:<br>
</div>
<div>
<div>
<blockquote type="cite">
<div dir="ltr">Hi Michael,
<div><br>
</div>
<div>why didn't you take my pull
request, modified it and then
merged it with the master
release? </div>
<div>From a contribution point of
view it should be clear where the
contributions came from and from
which pull request.</div>
<div>Now that history is not visible
in the yanel master branch.</div>
<div><br>
</div>
<div>Cheers, Balz</div>
</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Tue, Jun
7, 2016 at 11:16 PM, Michael
Wechner <span dir="ltr"><<a href="mailto:michael.wechner@wyona.com" target="_blank"></a><a href="mailto:michael.wechner@wyona.com" target="_blank">michael.wechner@wyona.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
<div>Hi Balz<br>
<br>
I am currently reviewing<br>
<br>
<a href="https://github.com/wyona/yanel/pull/80" target="_blank">https://github.com/wyona/yanel/pull/80</a><br>
<br>
Can you close<br>
<br>
<a href="https://github.com/wyona/yanel/pull/75" target="_blank">https://github.com/wyona/yanel/pull/75</a><br>
<br>
?<br>
<br>
Thanks<br>
<br>
Michael<br>
<br>
Am 07.06.16 um 22:57 schrieb
Michael Wechner:<br>
</div>
<blockquote type="cite">
<div>Hi Balz<br>
<br>
I have implemented "this",
whereas I have made it
configurable for the
redirect resource with
version "1.0.2", e.g.<br>
<br>
<yanel:resource-config
xmlns:yanel=<a href="http://www.wyona.org/yanel/rti/1.0" target="_blank"></a><a href="http://www.wyona.org/yanel/rti/1.0" target="_blank">"http://www.wyona.org/yanel/rti/1.0"</a>><br>
<yanel:rti
name="redirect" namespace=<a href="http://www.wyona.org/yanel/resource/1.0.2" target="_blank"></a><a href="http://www.wyona.org/yanel/resource/1.0.2" target="_blank">"http://www.wyona.org/yanel/resource/1.0.2"</a>/><br>
<br>
<yanel:property
name="href"
value="en/index.html"/><br>
<!-- INFO: We use 307
instead 301, such that the
redirect is always
checked, in case the
redirect location should
change in the future
--><br>
<yanel:property
name="status-code"
value="307"/><br>
<br>
<br>
which means you can set it
now to 301 or whatever.<br>
<br>
If you don't set it, then
the default is still 307
for backwards
compatibility reasons,
whereas I double checked
that to be sure that this
makes sense.<br>
<br>
If you think the
configurable version is
not enough or that the
default should be 301,
then please let me know,
because we could discuss
to introduce a new
redirect version, e.g.
"redirect, 2.0.0" which
uses 301.<br>
<br>
HTH<br>
<br>
Michael <br>
<br>
Am 03.06.16 um 12:58
schrieb basZero:<br>
</div>
<blockquote type="cite">
<div dir="ltr">Hi,
<div><br>
</div>
<div>the easiest of the
four pull requests I
sent some time ago is
this one:</div>
<div><a href="https://github.com/wyona/yanel/pull/75" target="_blank"></a><a href="https://github.com/wyona/yanel/pull/75" target="_blank">https://github.com/wyona/yanel/pull/75</a><br>
</div>
<div><br>
</div>
<div>It would be really
great if these could
be applied asap before
they become
incompatible again
(like those I sent
years ago).</div>
<div><br>
</div>
<div>Cheers, Balz</div>
</div>
<br>
<fieldset></fieldset>
<br>
</blockquote>
<br>
</blockquote>
<br>
</div>
<br>
--<br>
Yanel-development mailing list <a href="mailto:Yanel-development@wyona.com" target="_blank"></a><a href="mailto:Yanel-development@wyona.com" target="_blank">Yanel-development@wyona.com</a><br>
<a href="http://mx2.wyona.com/cgi-bin/mailman/listinfo/yanel-development" rel="noreferrer" target="_blank">http://mx2.wyona.com/cgi-bin/mailman/listinfo/yanel-development</a><br>
</blockquote>
</div>
<br>
</div>
<br>
<fieldset></fieldset>
<br>
</blockquote>
<br>
</div>
</div>
</div>
<br>
--<br>
Yanel-development mailing list <a href="mailto:Yanel-development@wyona.com" target="_blank"></a><a href="mailto:Yanel-development@wyona.com" target="_blank">Yanel-development@wyona.com</a><br>
<a href="http://mx2.wyona.com/cgi-bin/mailman/listinfo/yanel-development" rel="noreferrer" target="_blank">http://mx2.wyona.com/cgi-bin/mailman/listinfo/yanel-development</a><br>
</blockquote>
</div>
<br>
</div>
<br>
<fieldset></fieldset>
<br>
</blockquote>
<br>
</div>
</div>
</div>
<br>
--<br>
Yanel-development mailing list <a href="mailto:Yanel-development@wyona.com" target="_blank">Yanel-development@wyona.com</a><br>
<a href="http://mx2.wyona.com/cgi-bin/mailman/listinfo/yanel-development" rel="noreferrer" target="_blank">http://mx2.wyona.com/cgi-bin/mailman/listinfo/yanel-development</a><br>
</blockquote>
</div>
<br>
</div>
<br>
<fieldset></fieldset>
<br>
</blockquote>
<br>
</div></div></div>
<br>--<br>
Yanel-development mailing list <a href="mailto:Yanel-development@wyona.com">Yanel-development@wyona.com</a><br>
<a href="http://mx2.wyona.com/cgi-bin/mailman/listinfo/yanel-development" rel="noreferrer" target="_blank">http://mx2.wyona.com/cgi-bin/mailman/listinfo/yanel-development</a><br></blockquote></div><br></div>