Skip to content

Send host header to Decoupled Router. #856

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.

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

longwave
Copy link

@longwave longwave commented Jun 5, 2025

This pull request is for: (mark with an "x")

  • examples/*
  • modules/next
  • packages/next-drupal
  • starters/basic-starter
  • starters/graphql-starter
  • starters/pages-starter
  • Other

GitHub Issue: #332
Please add a link to the GitHub issue
where this problem is discussed.

  • I need help adding tests. (mark with an "x")
    Code changes need test coverage. If you don't know
    how to make tests, check this box to ask for help.

Describe your changes

This is an alternative to the suggestion in #332, instead of needing different Drupal base URLs we pass on the host header from the original request as a query parameter to Decoupled Router. I also have an extended version of Decoupled Router that interprets that query parameter and passes it to Domain module, allowing decoupled multi-domain setups with a single Drupal backend URL.

Copy link

vercel bot commented Jun 5, 2025

@longwave is attempting to deploy a commit to the Chapter Three Team on Vercel.

A member of the Team first needs to authorize it.

@longwave
Copy link
Author

longwave commented Jun 5, 2025

Well, except this doesn't really work because it makes all requests dynamic as soon as you look at headers; it works in development mode but not in a production build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant