Skip to content

fix links issue when pathPrefix is used#504

Draft
fblackburn1 wants to merge 4 commits intomasterfrom
fix-links-with-pathprefix
Draft

fix links issue when pathPrefix is used#504
fblackburn1 wants to merge 4 commits intomasterfrom
fix-links-with-pathprefix

Conversation

@fblackburn1
Copy link
Copy Markdown
Member

@fblackburn1 fblackburn1 commented Aug 10, 2022

why: pathPrefix option is used by zuul
To test with prefix

diff --git a/docker-compose-tests.yml b/docker-compose-tests.yml
index f968be7..b4bafa2 100644
--- a/docker-compose-tests.yml
+++ b/docker-compose-tests.yml
@@ -7,7 +7,7 @@ services:
       - ./public:/app/public:ro
     expose:
       - "8000"
-    command: bash -c "cd /app/public && python -m http.server 8000"
+    command: bash -c "cd /app && python -m http.server 8000"
   # Allow to run scrap tests before deployment
   test:
     build: tests
diff --git a/gatsby-config.js b/gatsby-config.js
index 6a3c341..0456e66 100644
--- a/gatsby-config.js
+++ b/gatsby-config.js
@@ -7,6 +7,7 @@ const path = require('path');
  */
 
 module.exports = {
+  pathPrefix: '/public',
   plugins: [
     'gatsby-plugin-react-helmet',
     {

@wazo-community-zuul
Copy link
Copy Markdown
Contributor

Build succeeded.

@fblackburn1 fblackburn1 marked this pull request as draft August 11, 2022 11:58
@sduthil sduthil force-pushed the fix-links-with-pathprefix branch from 106da5a to 225999c Compare August 11, 2022 13:09
@wazo-community-zuul
Copy link
Copy Markdown
Contributor

Build succeeded.

@wazo-community-zuul
Copy link
Copy Markdown
Contributor

Build failed.

@fblackburn1 fblackburn1 force-pushed the fix-links-with-pathprefix branch from 5edf750 to b95b2f0 Compare August 11, 2022 14:35
@wazo-community-zuul
Copy link
Copy Markdown
Contributor

Build succeeded.

@wazo-community-zuul
Copy link
Copy Markdown
Contributor

Build succeeded.

why: This component already handle withPrefix automatically
@fblackburn1 fblackburn1 force-pushed the fix-links-with-pathprefix branch from 8b9f40f to 92c7b25 Compare August 11, 2022 18:53
@fblackburn1
Copy link
Copy Markdown
Member Author

fblackburn1 commented Aug 11, 2022

With latest commits I have the following links broken when using prefix (see first post to how to test)

wazo-platformorg-test-1  |  Error in http://server:8000/public/tutorials (from http://server:8000/public/blog): Error: statuses 404                                                                        
wazo-platformorg-test-1  | Error in http://server:8000/uc-doc/system/wazo-confd/developer (from http://server:8000/public/documentation/overview/configuration.html): Error: statuses 404                  
wazo-platformorg-test-1  | Error in http://server:8000/contribute/code (from http://server:8000/public/contribute): Error: statuses 404                                                                    
wazo-platformorg-test-1  | Error in http://server:8000/uc-doc/installation/install-system (from http://server:8000/public/contribute): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/documentation (from http://server:8000/public/contribute): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/uc-doc/contributors/provisioning/developing_plugins (from http://server:8000/public/contribute): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/ecosystem (from http://server:8000/public/contribute): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/use-cases (from http://server:8000/public/blog/release-review-2211): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/uc-doc/upgrade/ (from http://server:8000/public/blog/release-review-2211): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/public/blog/creating-our-webrtc-demo (from http://server:8000/public/blog): Error: statuses 404,404,404,404,404,404,404,404
wazo-platformorg-test-1  | Error in http://server:8000/public/blog/creating-cross-platform-push-to-talk-app (from http://server:8000/public/blog): Error: statuses 404,404,404,404,404,404,404,404,404,404
wazo-platformorg-test-1  | Error in http://server:8000/uc-doc/upgrade/introduction (from http://server:8000/public/blog/release-review-2004): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/uc-doc/upgrade/upgrade_notes (from http://server:8000/public/blog/release-review-2003): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/blog/wazo-platform-c4-overview (from http://server:8000/public/blog/install-wazo-c4-with-ansible): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/use-cases/class-4 (from http://server:8000/public/blog/install-wazo-c4-with-ansible): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/documentation/overview/router-confd.html (from http://server:8000/public/blog/sprint-review-2001): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/public/blog/scale-asterisk-apps (from http://server:8000/public/blog): Error: statuses 404,404
wazo-platformorg-test-1  | Error in http://server:8000/public/blog/kamailio-developpers-meeting-2019-kemi (from http://server:8000/public/blog): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/blog/wazo-platform-c4-on-kubernetes (from http://server:8000/public/blog/kamailio-consul-service-discovery): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/public/blog/kamailio-ha-dispatcher-and-dmq (from http://server:8000/public/blog): Error: statuses 404
wazo-platformorg-test-1  | Error in https://www.queuemetrics.com/ (from http://server:8000/public/blog/wazo-queuemetrics-partnership): Error: statuses 403
wazo-platformorg-test-1  | Error in http://server:8000/blog/wazo-admin-ui (from http://server:8000/public/blog/sprint-review-1707): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/blog/sprint-review-1616 (from http://server:8000/public/blog/wazo-admin-ui): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/blog/sprint-review-1701 (from http://server:8000/public/blog/wazo-admin-ui): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/blog/sprint-review-1706 (from http://server:8000/public/blog/wazo-admin-ui): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/blog/introducing-wazo (from http://server:8000/public/blog/sprint-review-1616): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/public/uc-doc/installation/install-system (from http://server:8000/public/use-cases): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/public/uc-doc/ (from http://server:8000/public/use-cases): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/uc-doc/contributors/provisioning/introduction-to-the-plugin-model-of-the-new-provisioning-server (from http://server:8000/public/documentation/overview/provisioning.html): Error: statuses 404
wazo-platformorg-test-1  | Error in http://server:8000/uc-doc/contributors/provisioning/httptftp-requests-processing-in-provd-part-1 (from http://server:8000/public/documentation/overview/provisioning.html): Error: statuses 404          
wazo-platformorg-test-1  | Error in http://server:8000/uc-doc/contributors/provisioning/httptftp-requests-processing-in-provd-part-2 (from http://server:8000/public/documentation/overview/provisioning.html): Error: statuses 404

@wazo-community-zuul
Copy link
Copy Markdown
Contributor

Build succeeded.

@sduthil
Copy link
Copy Markdown
Contributor

sduthil commented Aug 24, 2022

recheck

@wazo-community-zuul
Copy link
Copy Markdown
Contributor

Build failed.

@sduthil
Copy link
Copy Markdown
Contributor

sduthil commented Aug 24, 2022

recheck

@wazo-community-zuul
Copy link
Copy Markdown
Contributor

Build failed.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants