moonming commented on PR #2086: URL: https://github.com/apache/apisix-website/pull/2086#issuecomment-5104344026
Independent audit follow-up for `cc4d35d47`: - **HIGH - ingress-controller version/source mismatch: resolved.** The page now labels `2.1.0` as the latest project version and separately exposes the currently available ASF source `1.8.0`; the archive path is `apisix/ingress-controller/1.8.0/apache-apisix-ingress-controller-1.8.0-src` and the real `.tgz` URL returns HTTP 200. - **MEDIUM - E2E ran before the deployment overlay: resolved.** Playwright now runs after Astro is copied into `website/build`, serves that final tree, and verifies both migrated routes and retained Docusaurus routes in desktop/mobile projects. - **MEDIUM - plugin icon fidelity: resolved.** The canonical sprite is copied from `website/static/js/plugin-icon.js`, and cards again use uploaded logos, the APISIX default icon, or the plugin-specific sprite symbol. - **MEDIUM - social preview regression: resolved.** The production APISIX preview image is now local, and migrated pages always emit `og:image`, `twitter:image`, and `summary_large_image`. - **LOW - nested main landmark: resolved.** The plugin catalog uses a labelled `section`; E2E asserts exactly one `main`. Verification: Astro build produced 924 pages; content/link checks report 0 errors and 0 broken doc links; migrated-page Playwright scenarios pass at 1440x900 and 390x844; retained Docusaurus route checks pass in both viewports against the live hybrid site. -- 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]
