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.

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

DOP-3998: allow preprd to use URL for search staging server #937

Closed
wants to merge 5 commits into from

Conversation

seungpark
Copy link
Contributor

@seungpark seungpark commented Nov 16, 2023

PR description:

  • this PR allows autobuilder to pipe in a front end variable GATSBY_MARIAN_URL to let the client side know which search server to point to.
  • AWS parameter store values have been stored, with stg environment pointing to stage search url https://docs-search-transport.docs.staging.corp.mongodb.com/

image

image

PREREQ prs
search transport server must allow CORS authentication
front end must supply credentials when making requests to authenticate through corpsecure

Sample of change
Staged site, making requests to staging search server (will have to be authenticated in same browser to make send credentials)

Copy link

The URL for your feature branch webhook is https://15qnqwga12.execute-api.us-east-2.amazonaws.com/prod/

1 similar comment
Copy link

The URL for your feature branch webhook is https://15qnqwga12.execute-api.us-east-2.amazonaws.com/prod/

@@ -23,7 +23,7 @@ RUN cd ./modules/oas-page-builder \
FROM ubuntu:20.04
ARG WORK_DIRECTORY=/home/docsworker-xlarge
ARG SNOOTY_PARSER_VERSION=0.15.0
ARG SNOOTY_FRONTEND_VERSION=0.15.1.1b
ARG SNOOTY_FRONTEND_VERSION=DOP-3998
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: revert this change. testing with front end changes in feature branch stack

@seungpark
Copy link
Contributor Author

moving to new PR

@seungpark seungpark closed this Nov 21, 2023
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