Skip to content

docs(blog): fix CSP issue by replacing shields.io badge with local image#872

Merged
delei merged 2 commits intoapache:mainfrom
bengbengbalabalabeng:fix-docs-csp
Feb 28, 2026
Merged

docs(blog): fix CSP issue by replacing shields.io badge with local image#872
delei merged 2 commits intoapache:mainfrom
bengbengbalabalabeng:fix-docs-csp

Conversation

@bengbengbalabalabeng
Copy link
Contributor

Purpose of the pull request

What's changed?

Replaced the external Shields.io badge in the Welcome New Committer blog post with a local image. The previous external image was blocked by the site’s Content Security Policy. Using a local asset ensures the badge loads correctly.

Before:

image

After:

image

Checklist

  • I have read the Contributor Guide.
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

@bengbengbalabalabeng bengbengbalabalabeng changed the title docs: fix CSP issue by replacing shields.io badge with local image re… docs: fix CSP issue by replacing shields.io badge with local image Feb 27, 2026
@bengbengbalabalabeng bengbengbalabalabeng changed the title docs: fix CSP issue by replacing shields.io badge with local image docs(blog): fix CSP issue by replacing shields.io badge with local image Feb 27, 2026
@delei
Copy link
Member

delei commented Feb 27, 2026

Hi, @GOODBOY008

Is an image file missing :) ?

@GOODBOY008
Copy link
Member

GOODBOY008 commented Feb 28, 2026

Hi, @GOODBOY008

Is an image file missing :) ?

Because the securety ban the external resource load. @delei
企业微信截图_d0ea1db3-1a57-49f1-aa33-03caa2da4f5f

---

![Apache Fesod New Committer](https://img.shields.io/badge/Apache%20Fesod-New%20Committer-blue?style=for-the-badge&logo=apache)
![Apache Fesod New Committer](../../../static/img/blog/apache-fesod-new-committer.svg)
Copy link
Member

Choose a reason for hiding this comment

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

Please do not use relative paths.

Suggested change
![Apache Fesod New Committer](../../../static/img/blog/apache-fesod-new-committer.svg)
![Apache Fesod New Committer](/img/blog/apache-fesod-new-committer.svg)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Thanks for the suggestion :)

Copy link
Member

@delei delei left a comment

Choose a reason for hiding this comment

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

LGTM

@delei delei merged commit 912a8de into apache:main Feb 28, 2026
5 checks passed
@bengbengbalabalabeng bengbengbalabalabeng deleted the fix-docs-csp branch March 1, 2026 06:07
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.

3 participants