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

feat(gatsby-source-drupal): Provide proxyUrl in addition to baseUrl to allow using CDN, API gateway, etc. #36819

Merged
merged 19 commits into from
Nov 22, 2022
Merged

feat(gatsby-source-drupal): Provide proxyUrl in addition to baseUrl to allow using CDN, API gateway, etc. #36819

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
5427cfc
[gatsby-source-drupal] provide proxyUrl in addition to baseUrl to all…
Vacilando Oct 13, 2022
2cd261c
[gatsby-source-drupal] provide proxyUrl in addition to baseUrl to all…
Vacilando Oct 13, 2022
d6d63af
[gatsby-source-drupal] provide proxyUrl in addition to baseUrl to all…
Vacilando Oct 13, 2022
9314836
[gatsby-source-drupal] improve README.me about the proxyUrl param
Vacilando Oct 13, 2022
0cb00ba
[gatsby-source-drupal] remove debug commands
Vacilando Oct 13, 2022
7601aff
Update packages/gatsby-source-drupal/README.md
Vacilando Oct 13, 2022
b125620
[gatsby-source-drupal] fixing lint based on circleci feedback
Vacilando Oct 13, 2022
6ed9812
Update packages/gatsby-source-drupal/README.md
Vacilando Oct 13, 2022
ab070eb
Update README.md
Vacilando Oct 14, 2022
4cd98ea
Merge branch 'gatsbyjs:master' into master
Vacilando Oct 14, 2022
8e2cebc
Merge branch 'gatsbyjs:master' into master
Vacilando Nov 8, 2022
1484d0f
Merge branch 'master' into master
Vacilando Nov 18, 2022
b548340
Merge branch 'master' into pr/36819
TylerBarnes Nov 22, 2022
bf5b0b1
add proxy url test
TylerBarnes Nov 22, 2022
837f9a3
expect got to be called with string containing proxy url
TylerBarnes Nov 22, 2022
8bb2774
expect each got call to have proxyUrl except for the first one
TylerBarnes Nov 22, 2022
36dec91
Update index.js
TylerBarnes Nov 22, 2022
7421b2d
make sure we only skip once
TylerBarnes Nov 22, 2022
c200fec
Update index.js
TylerBarnes Nov 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update packages/gatsby-source-drupal/README.md 
Co-authored-by: Kyle Mathews <mathews.kyle@gmail.com>
  • Loading branch information
@Vacilando @KyleAMathews
Vacilando and KyleAMathews committed Oct 13, 2022
commit 6ed98124c7c23a81c2e890c99a63fe4e751b7fbe
2 changes: 2 additions & 0 deletions packages/gatsby-source-drupal/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -255,6 +255,8 @@ module.exports = {
You can add an optional CDN or API gateway URL `proxyUrl` param. The URL can be a simple proxy of the Drupal
`baseUrl`, or another URL (even containing a path) where the Drupal JSON API resources can be retrieved.
Vacilando marked this conversation as resolved.
Show resolved Hide resolved

This option is required as Drupal doesn't know about the CDN so it returns URLs pointing to the `baseUrl`. With `proxyUrl` set, the plugin will rewrite URLs returned from Drupal to keep pointing at the `proxyUrl`

```javascript
// In your gatsby-config.js
module.exports = {
Expand Down