Skip to content

Fix zpages CSS and JS CDN URLs#8502

Merged
MrAlias merged 5 commits into
open-telemetry:mainfrom
pradhyum6144:fix-zpages-css-cdn
Feb 4, 2026
Merged

Fix zpages CSS and JS CDN URLs#8502
MrAlias merged 5 commits into
open-telemetry:mainfrom
pradhyum6144:fix-zpages-css-cdn

Conversation

@pradhyum6144
Copy link
Copy Markdown
Contributor

Fixes #8470

Problem
The zpages module references Material Design Lite CSS and JavaScript from code.getmdl.io, which is currently returning 403 Forbidden errors, breaking the zpages UI.

Solution
Replaced the broken CDN with cdnjs.cloudflare.com, which is actively maintained and accessible.

Changes

  • Updated zpages/internal/templates/header.html to use cdnjs.cloudflare.com CDN URLs
  • Both CSS and JS URLs have been verified to return HTTP 200
  • Added CHANGELOG entry under "Fixed" section

Testing

  • Verified new CDN URLs are accessible (HTTP 200)
  • All zpages tests pass: go test -race ./zpages/...
  • No functional changes to the code, only CDN URLs updated

Verification

$ curl -I https://cdnjs.cloudflare.com/ajax/libs/material-design-lite/1.3.0/material.indigo-pink.min.css
HTTP/2 200

$ curl -I https://cdnjs.cloudflare.com/ajax/libs/material-design-lite/1.3.0/material.min.js
HTTP/2 200

Signed-off-by: pradhyum6144 <pradhyum314@gmail.com>
Signed-off-by: pradhyum6144 <pradhyum314@gmail.com>
@pradhyum6144 pradhyum6144 requested review from a team and dashpole as code owners February 3, 2026 19:21
@pradhyum6144
Copy link
Copy Markdown
Contributor Author

Hello @dashpole sir, can review this if there is any changes need mention i will change or please review and merge it .
Thank you sir.

Copy link
Copy Markdown
Contributor

@dashpole dashpole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@pradhyum6144
Copy link
Copy Markdown
Contributor Author

Thanks!

Thank you sir for assigning this issue can you please merge it. If there are any issues please assign me i am ready to work.

@dashpole
Copy link
Copy Markdown
Contributor

dashpole commented Feb 3, 2026

This requires a second review.

Comment thread zpages/internal/templates/header.html
Comment thread CHANGELOG.md Outdated
Copy link
Copy Markdown
Contributor

@MrAlias MrAlias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM outside of the changelog entry needed fix.

Signed-off-by: pradhyum6144 <pradhyum314@gmail.com>
@MrAlias
Copy link
Copy Markdown
Contributor

MrAlias commented Feb 3, 2026

@pradhyum6144 the only requirement this PR still needs to satisfy is to remain open for a full day. Thanks for the contribution.

@MrAlias MrAlias merged commit 98f36e2 into open-telemetry:main Feb 4, 2026
29 checks passed
@MrAlias MrAlias added this to the v1.41.0 milestone Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

zpages: CSS is not accessible

4 participants