Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Canonical URLs #342

Closed
GoogleCodeExporter opened this issue Mar 25, 2015 · 12 comments
Closed

Canonical URLs #342

GoogleCodeExporter opened this issue Mar 25, 2015 · 12 comments

Comments

@GoogleCodeExporter
Copy link

http://googlewebmastercentral.blogspot.com/2009/02/specify-your-canonical.html

<link rel="canonical" href=" ... current url without seaside fields ... " /> 

Original issue reported on code.google.com by renggli on 13 Feb 2009 at 2:33

@GoogleCodeExporter
Copy link
Author

I gave a try fro 2.9... not sure at all this is the correct way though.

I put it in WAHtmlRoot>>writeHeadOn: 
Maybe this is problematic if we want to overide the defaut (in updateRoot?)?

Relative path seems to be ok according to the website [1] but now, parameters 
and the
fragment is not taken into account (don't know if they have too anyway).

Cheers,

Cédrick

[1]
"Can I use a relative path to specify the canonical, such as <link 
rel="canonical"
href="product.php?item=swedish-fish" />?
Yes, relative paths are recognized as expected with the <link> tag. Also, if you
include a <base> link in your document, relative paths will resolve according 
to the
base URL. "

Original comment by cdric...@gmail.com on 13 Feb 2009 at 4:25

  • Added labels: ****
  • Removed labels: ****

Attachments:

@GoogleCodeExporter
Copy link
Author

I'm not convinced this should happen automatically. I think we should just 
provide
helper infrastructure for the applications that care about URLs and have custom 
URL
handling in place.

Original comment by philippe...@gmail.com on 13 Feb 2009 at 4:35

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Why not, is there a disadvantage of not adding it? Do you see cases where 
adding a default canonical tag would 
be wrong? People will still have to implement #updateUrl: and #initialRequest: 
if they want to make pages being 
properly indexed, otherwise the canonical URL will just point to the entry 
point.

Maybe in the future browser will also pick up that tag and use it to cleanly 
bookmark sites and copy URLs?

Original comment by renggli on 13 Feb 2009 at 7:11

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Because there is no way for Seaside to know what the canonical URL is. It is 
certainly not the default application URL. Because per default that content is 
very 
much dependent of the _s and _k values. So the cannonical URL is the URL with 
_s and 
_k with makes the whole exercise pointless. Only slight differences are allowed 
for 
the contents and we violate this if we exclude _s and _k.

And hardcoding it into WAHtmlRoot so that it can not be customized and only be 
used 
when there is a request certainly doesn't imporve the situation.

I believe it's an optimization you want make when you do URL optimizations and 
SEO 
in general. Only you know about your URLs and only you can make the right trade 
offs.

Original comment by philippe...@gmail.com on 13 Feb 2009 at 7:42

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

You are missing the point.

There is no point in including _s and _k, because they are random. Never ever. 
There is no deterministic 
connection between these two parameters and the page contents.

Anything that can be bookmarked or that is valuable to a search engine is added 
through #updateUrl:. I can't see 
a single case where you want the canonical URL be something different than what 
you get through #updateUrl: 
minut _s and _k.

Original comment by renggli on 13 Feb 2009 at 8:42

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

"canonical" being an /optional/ element, wouldn't it's canonical place be a 
decoration?

Original comment by stefan.s...@gmail.com on 13 Feb 2009 at 9:30

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

So we can't take _s and _k and we can't take the default URL. We can't enable 
it by 
default.

Original comment by philippe...@gmail.com on 14 Feb 2009 at 1:07

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Everybody can bookmark an application anytime. Even if this is not supported by 
the application. To what would 
the canonical URL be different?

Original comment by renggli on 14 Feb 2009 at 1:11

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

I have to agree with philippe on this one. I'm not convinced this should happen
automatically. If nothing else, it's just crud and more code for internal sites 
that
will never be indexed by Google.

I'm also not convinced that the entry point URL without the _k and _s is a good
default for applications that have not set up their site to use #updateUrl:, 
etc.
Google might penalize sites that are returning very different pages (which these
would be) as their canonical URLs.

Original comment by jfitz...@gmail.com on 14 Feb 2009 at 6:54

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Ok, you convinced me. I close this issue then. Happy URL fiddling ;-)

Original comment by renggli on 14 Feb 2009 at 7:01

  • Changed state: WontFix
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

I included the #beCanonical though:

Name: Seaside-Core-lr.464
Author: lr
Time: 14 February 2009, 8:06:45 pm
UUID: 57db9aef-3add-4e2a-9b3b-0b19926f5fdf
Ancestors: Seaside-Core-jf.463

- integrated WALinkElement-beCanonical.st

Original comment by renggli on 14 Feb 2009 at 7:07

  • Changed state: Fixed
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Original comment by renggli on 20 Feb 2009 at 11:23

  • Added labels: ****
  • Removed labels: ****

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant