<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><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><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">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">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">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 class="h5">
<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">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">"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">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">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>