olsoloviov commented on code in PR #2939:
URL: https://github.com/apache/polaris/pull/2939#discussion_r2484898593


##########
site/content/in-dev/unreleased/helm.md:
##########
@@ -22,36 +22,22 @@ type: docs
 weight: 675

Review Comment:
   Thanks @MonkeyCanCode, I did not know it was automatically copied. But I see 
that there are already merged changes that diverge helm.md from 
helm/polaris/readme.md - markup for blockquotes was changed in 
https://github.com/apache/polaris/pull/2656.
   Also, there is a problem with rendering badges in the original page header 
when hosted on the apache.org:
   <img width="809" height="112" alt="image" 
src="https://github.com/user-attachments/assets/d0e2d086-8c49-4862-882e-7412ed5ce8dc";
 />
   Apparently apache.org content security policy does not allow embedding 
images from external hosts:
   ```
   Refused to load the image 
'https://img.shields.io/badge/Version-1.2.0--incubating--SNAPSHOT-informational?style=flat-square'
 because it violates the following Content Security Policy directive: 
"default-src 'self' data: blob: 'unsafe-inline' 'unsafe-eval' 
https://www.apachecon.com/ https://www.communityovercode.org/ 
https://*.apache.org/ https://apache.org/ https://*.scarf.sh/";. Note that 
'img-src' was not explicitly set, so 'default-src' is used as a fallback.
   ```
   It was my motivation to remove the header from page version for the site.
   
   As a sidenote, we have other pages on the site that are copies from 
getting-started docs (e.g. Keycloak setup  page, Telemetry setup page), but 
they are not copied automatically. It would be good to have some unified 
approach for such cases. 



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to