Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix links issue when pathPrefix is used #504

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

fix links issue when pathPrefix is used #504

wants to merge 4 commits into from

Conversation

fblackburn1
Copy link
Member

@fblackburn1 fblackburn1 commented Aug 10, 2022 •
edited
Loading

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
Contributor

Build succeeded.

@fblackburn1 fblackburn1 marked this pull request as draft August 11, 2022 11:58
@wazo-community-zuul
Copy link
Contributor

Build succeeded.

@wazo-community-zuul
Copy link
Contributor

Build failed.

@wazo-community-zuul
Copy link
Contributor

Build succeeded.

@wazo-community-zuul
Copy link
Contributor

Build succeeded.

@fblackburn1
do not use withPrefix with Link component
92c7b25 
why: This component already handle withPrefix automatically
@fblackburn1
Copy link
Member Author

fblackburn1 commented Aug 11, 2022 •
edited
Loading

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
Contributor

Build succeeded.

@sduthil
Copy link
Member

sduthil commented Aug 24, 2022

recheck

@wazo-community-zuul
Copy link
Contributor

Build failed.

@sduthil
Copy link
Member

sduthil commented Aug 24, 2022

recheck

@wazo-community-zuul
Copy link
Contributor

Build failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
mergeit
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fblackburn1 @sduthil