You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2022/03/31 08:12:58 UTC
[GitHub] [apisix-website] nfrankel opened a new pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
nfrankel opened a new pull request #995:
URL: https://github.com/apache/apisix-website/pull/995
Duplicate content is frowned upon by Google. This blog post has been already published on the OpenTelemetry web site. To avoid being flagged as duplicate and incur a SEO penalty, we need to reference the "original".
Alternatively, we could ask the OpenTelemetry team to point to our post. But it will probably take more time and it's not worth it IMHO.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084724340
https://github.com/open-telemetry/opentelemetry.io/pull/1254#discussion_r839675423 solves part of our concerns in this issue.
This pull request will solve the rest of our concerns.
https://github.com/open-telemetry/opentelemetry.io/pull/1258
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 merged pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 merged pull request #995:
URL: https://github.com/apache/apisix-website/pull/995
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084277029
@yzeng25 Technically, I'm afraid that Google might disagree with you π
> Duplicate content generally refers to substantive blocks of content within or across domains that either completely match other content in the same language or **are appreciably similar**
>
>-- [Avoid creating duplicate content](https://developers.google.com/search/docs/advanced/guidelines/duplicate-content)
(Emphasis is mine)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084274524
@yzeng25 No. Canonical Tags mean which URL should be indexed. If merged this, https://apisix.apache.org/blog/2022/02/28/apisix-integration-opentelemetry-plugin may never be indexed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084281262
>we can read the [HTML Standard](https://html.spec.whatwg.org/multipage/links.html#link-type-canonical)
Unfortunately, it has nothing to do with HTML standard but everything to do with how Google indexes documents
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084265176
@SkyeYoung It makes sense. Are you willing to ask (and follow up) the OpenTelemetry team to update their page?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084329223
> @yzeng25 Nope
>
> > This is the source repo for the website and project documentation. The site is using **Hugo** and hosted on Netlify.
> > -- https://github.com/open-telemetry/opentelemetry.io
>
> We need to understand:
>
> 1. If we can override the canonical link with Hugo
> 2. How to do it
There you go, I think it is totally feasible.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084354510
@nfrankel @SkyeYoung I have it done [here](https://github.com/open-telemetry/opentelemetry.io/pull/1254/commits/cda39aab4149ebd5d18539e21939a776aaa5a749). Normally it takes a couple of days. I will track it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085685173
> > 2. We should review GA stats after a month or so, to see if `canonical_url` actually works. If it works, then we should do it for each cross-posted blog that is posted on platforms other than Medium. If it does not work, we should rollback the change in this pull request.
>
> Let's create an issue to track this.
@nfrankel @SkyeYoung , just created an issue to track this: https://github.com/apache/apisix-website/issues/999. We can discuss over there.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084336913
>They don't seem to like following their own standards
They don't have "standards". They are Google. They do whatever they want.
>Maybe we don't have to do anything
Definitely no. I'm not an SEO expert, but I've dabbled a bit into it, and duplicate content is just plain bad.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085484228
@yzeng25 Maybe they just have not intended to support Canonical Tag. π€
I remember that hugo did not have related FrontMatter (I used to be a user of it)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084279966
>Canonical Tags mean which URL should be indexed
Can you please provide references to this statement @SkyeYoung
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085427808
Sorry, but I don't see any canonical link in the deployed page...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085475517
Ok. Take Medium as an example, after adding canonical_url, we are expecting something like this to show up.
I checked all blog posts on OpenTelemetry's site, all blogs with `canonical_url` are not reflecting similar changes as above.
So we need to go back to the second option, add canonical_url via this pull request, right?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084355512
Good job!
________________________________
εδ»ΆδΊΊ: yilinzeng ***@***.***>
ειζΆι΄: Thursday, March 31, 2022 5:58:51 PM
ζΆδ»ΆδΊΊ: apache/apisix-website ***@***.***>
ζι: SkyeYoung ***@***.***>; Mention ***@***.***>
δΈ»ι’: Re: [apache/apisix-website] fix: Explicitly set the rel=canonical of the OpenTelemetry blog post (PR #995)
@nfrankel<https://github.com/nfrankel> @SkyeYoung<https://github.com/SkyeYoung> I have it done here<https://github.com/open-telemetry/opentelemetry.io/pull/1254/commits/cda39aab4149ebd5d18539e21939a776aaa5a749>. Normally it takes a couple of days. I will track it.
β
Reply to this email directly, view it on GitHub<https://github.com/apache/apisix-website/pull/995#issuecomment-1084354510>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ALRIROFAZD63G4VPWO7CTKDVCVZNXANCNFSM5SERCO4A>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084278822
@nfrankel @yzeng25 we can read the [HTML Standard](https://html.spec.whatwg.org/multipage/links.html#link-type-canonical)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085552994
@yzeng25 Please merge this PR. We tried, we failed. Now it's time to fix.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
netlify[bot] edited a comment on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084245292
### <span aria-hidden="true">β
</span> Deploy Preview for *apache-apisix* ready!
| Name | Link |
|---------------------------------|------------------------|
|<span aria-hidden="true">π¨</span> Latest commit | 09ecedcb155af7dfbd8b13506ae1a5eb4fd3b4a4 |
|<span aria-hidden="true">π</span> Latest deploy log | https://app.netlify.com/sites/apache-apisix/deploys/6245627dc40a2a00086627c9 |
|<span aria-hidden="true">π</span> Deploy Preview | https://deploy-preview-995--apache-apisix.netlify.app |
|<span aria-hidden="true">π±</span> Preview on mobile | <details><summary> Toggle QR Code... </summary><br /><br /><br /><br />_Use your smartphone camera to open QR code link._</details> |
---
_To edit notification comments on pull requests, go to your [Netlify site settings](https://app.netlify.com/sites/apache-apisix/settings/deploys#deploy-notifications)._
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085634106
> 2\. We should review GA stats after a month or so, to see if `canonical_url` actually works. If it works, then we should do it for each cross-posted blog that is posted on platforms other than Medium. If it does not work, we should rollback the change in this pull request.
Let's create an issue to track this.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084301313
>Authors who declare the canonical link relation ought to anticipate that applications such as search engines **can**:
>
>* Index content only from the target IRI (i.e., content from the context IRIs will be likely disregarded as duplicative).
(Emphasis mine).
@SkyeYoung If I understand correctly, your point is that if we set the `rel=canonical`, our page **may** not be indexed.
My point is that I don't care, because it **will** be flagged as duplicate by Google, thus our domain **will** lose PageRank or whatever secret metric Google uses nowadays and our page **may** not be indexed anyway.
I stand by my position: we need to set the `rel=canonical`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084262627
I don't think it's a good idea. Canonical Tags specify the URL that should be shown in search results. It should be used to specify the version of a page in our website, not other websites.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084342011
> They don't have "standards". They are Google. They do whatever they want.
So I use DDG, but not Google. :p
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084348905
@yzeng25 yes, if it's merged fast
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085630783
It is a completely new case for me. As we discussed, tried and encountered failures, let's agree to disagree.
1. I would like to get this pull request merged. Giving it a shot wouldn't hurt.
2. We should review GA stats after a month or so, to see if `canonical_url` actually works. If it works, then we should do it for each cross-posted blog that is posted on platforms other than Medium. If it does not work, we should rollback the change in this pull request.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084304233
The best would definitely be to add the canonical on their side and point to our blog (as I mentioned in the PR description).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084347192
> > @yzeng25 Nope
> > > This is the source repo for the website and project documentation. The site is using **Hugo** and hosted on Netlify.
> > > -- https://github.com/open-telemetry/opentelemetry.io
> >
> >
> > We need to understand:
> >
> > 1. If we can override the canonical link with Hugo
> > 2. How to do it
>
> There you go, I think it is totally feasible.
>
> 
@nfrankel @SkyeYoung as I understand, this is the most satisfying way for all of us, right?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085495559
> Positive side: we found a bug for them.
Maybe this is the behavior they expect...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085283701
@nfrankel @SkyeYoung, since both https://github.com/open-telemetry/opentelemetry.io/pull/1258 and https://github.com/open-telemetry/opentelemetry.io/pull/1254 are merged now, canonical url is added on OpenTelemetry's side. See https://opentelemetry.io/blog/2022/apisix/#summary.
I think we can close this pull request.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085634299
@yzeng25 Perfect for me
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084313530
@yzeng25 I don't know if they are using Docusaurus too, but if they are, yes, something like that could work. But in the meanwhile, Google would flag us as duplicate. So it depends a lot on how fast they can merge the PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084369024
> Thanks a lot @yzeng25!
No problem. Do you mind converting this pull request to draft for now? I want to see how it goes on the other side first. In this way, this pull request won't get merged or need re-approval.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] EmilyKeer commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
EmilyKeer commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084255424
Looks good to me
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084286102
> > Canonical Tags mean which URL should be indexed
>
> Can you please provide references to this statement @SkyeYoung
@nfrankel Pls read https://datatracker.ietf.org/doc/html/rfc6596.
> Unfortunately, it has nothing to do with HTML standard but everything to do with how Google indexes documents
We may need to know the members of WHATWG. :D
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084302628
So is it better to add Canonical on OpenTelemetry's side? Or adding Canonical is a bad idea for both sides? @nfrankel @SkyeYoung
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084307098
> My point is that I don't care, because it **will** be flagged as duplicate by Google, thus our domain **will** lose PageRank or whatever secret metric Google uses nowadays and our page **may** not be indexed anyway.
**Duplicate** just say on the **same** website.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085452323
@yzeng25 @nfrankel It seems to be more like a variable, and not parsed to canonical tag
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084349603
@SkyeYoung Ah yes. Unfortunately (or not), Google is ubiquitous everywhere outside of China (and perhaps Russia with Yandex)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084343159
DDG?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] netlify[bot] commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
netlify[bot] commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084245292
### <span aria-hidden="true">π·</span> Deploy Preview for *apache-apisix* processing.
| Name | Link |
|---------------------------------|------------------------|
|<span aria-hidden="true">π¨</span> Latest commit | 09ecedcb155af7dfbd8b13506ae1a5eb4fd3b4a4 |
|<span aria-hidden="true">π</span> Latest deploy log | https://app.netlify.com/sites/apache-apisix/deploys/6245627dc40a2a00086627c9 |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084309332
> The best would definitely be to add the canonical on their side and point to our blog (as I mentioned in the PR description).
Say if I had a pull request opening in OpenTelemetry's side, and I'll add something like this to it.
```
`<head>
<link rel="canonical" href="https://apisix.apache.org/blog/2022/02/28/apisix-integration-opentelemetry-plugin//" />
</head>`
```
And that is it, right?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084319120
@nfrankel adding the above code block to this pull request should solve all problems, right?
https://github.com/open-telemetry/opentelemetry.io/pull/1254
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084345428
@nfrankel Duck Duck Go :D
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084311543
>Duplicate just say on the same website
No!!!
Please read my [above comment](https://github.com/apache/apisix-website/pull/995#issuecomment-1084277029). To make it easier, I'll again paste and emphasize the important bits:
>Duplicate content generally refers to substantive blocks of content within or **across domains** that either completely match other content in the same language or are appreciably similar
>
>-- [Duplicate content](https://developers.google.com/search/docs/advanced/guidelines/duplicate-content)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] SkyeYoung commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
SkyeYoung commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084333436
> Please read my [above comment](https://github.com/apache/apisix-website/pull/995#issuecomment-1084277029). To make it easier, I'll again paste and emphasize the important bits:
>
> > Duplicate content generally refers to substantive blocks of content within or **across domains** that either completely match other content in the same language or are appreciably similar
> > -- [Duplicate content](https://developers.google.com/search/docs/advanced/guidelines/duplicate-content)
Sorry, I didn't see that. Interesting. They don't seem to like following their own standards.
But, I found:
> Duplicate content on a site is not grounds for action on that site unless it appears that the intent of the duplicate content is to be deceptive and manipulate search engine results. If your site suffers from duplicate content issues, and you don't follow the advice listed in this document, we do a good job of choosing a version of the content to show in our search results.
>
> -- [Duplicate content](https://developers.google.com/search/docs/advanced/guidelines/duplicate-content)
Maybe we don't have to do anything
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084332525
Amazing @yzeng25! Then by all means, please open a PR. But let's watch it, and if it's not merged within a reasonable time frame, we shall get back to this one.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084362522
Thanks a lot @yzeng25!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085448436
> Sorry, but I don't see any canonical link in the deployed page...
It is here.
https://raw.githubusercontent.com/open-telemetry/opentelemetry.io/main/content/en/blog/2022/apisix.md
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084269507
Technically they are two articles with different titles and contents. Adding it wouldn't hurt, but I think it is also fine to leave it as it is.
https://opentelemetry.io/blog/2022/apisix/
https://apisix.apache.org/blog/2022/02/28/apisix-integration-opentelemetry-plugin
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084327118
I had a [quick look](https://gohugo.io/content-management/urls/) and it's a bit involved. It seems you cannot just add a snippet: you'd need to actually to [change the template](https://cconcannon.medium.com/adding-an-external-canonical-url-to-a-hugo-template-3b6e18814649).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
nfrankel commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1084322566
@yzeng25 Nope
>This is the source repo for the website and project documentation. The site is using **Hugo** and hosted on Netlify.
>
>-- https://github.com/open-telemetry/opentelemetry.io
We need to understand:
1. If we can override the canonical link with Hugo
2. How to do it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #995: fix: Explicitly set the rel=canonical of the OpenTelemetry blog post
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on pull request #995:
URL: https://github.com/apache/apisix-website/pull/995#issuecomment-1085488180
> @yzeng25 Maybe they just have not intended to support Canonical Tag. π€
>
> I remember that hugo did not have related FrontMatter (I used to be a user of it)
This is hilarious πγI spend hours to work on it and ask their community to merge pull requests, and it is not working as expected.
Positive side: we found a bug for them.
Negative side: we need an alternative to solve canonical_url issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org