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

Bug: og:url doesn't have complete top-level domain #7168

Closed
lukefretwell opened this issue Dec 21, 2023 · 3 comments · Fixed by #7563
Closed

Bug: og:url doesn't have complete top-level domain #7168

lukefretwell opened this issue Dec 21, 2023 · 3 comments · Fixed by #7563
Assignees

Comments

@lukefretwell
Copy link

lukefretwell commented Dec 21, 2023

Describe the bug
A clear and concise description of what the bug is.

It looks like the og:url doesn't include top-level domain which could pose problems for indexing and sharing.

How Digital.gov displays (doesn't include top-level domain):

<meta property="og:url" content="/resources/an-introduction-to-domain-management/" />

Ideal display (includes top-level domain):

<meta property="og:url" content="https://www.whitehouse.gov/briefing-room/speeches-remarks/2023/12/21/remarks-by-president-biden-after-marine-one-arrival-21/" />

Reference: https://ogp.me/#url

Steps to Reproduce
Steps to reproduce the behavior:

  1. Go to 'https://digital.gov/resources/an-introduction-to-domain-management/'
  2. Click on 'View source' in browser
  3. Scroll down to '<meta property="og:url" content="/resources/an-introduction-to-domain-management/" />'
  4. See error

Screenshots
If applicable, add screenshots to help explain your problem.

@nick-mon1
Copy link
Contributor

@lukefretwell Thanks for creating this issue, our team will add this to our backlog to address.

Copy link

Hello! 👋 It looks like this issue is stale because it has been open 60 days with no activity. This is usually because the request was already solved or it is no longer applicable. If this issue should remain open, remove the stale label or add a comment. Otherwise, this will be closed in 7 days.

@lukefretwell
Copy link
Author

@ToniBonittoGSA @nick-mon1 @RileySeaburg @mejiaj! 🇺🇸

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

Successfully merging a pull request may close this issue.

2 participants