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 #942

Merged
merged 7 commits into from
Nov 27, 2023

Conversation

seungpark
Copy link
Contributor

@seungpark seungpark commented Nov 21, 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 for search staging to send credentials)

generate authentication token by visiting https://docs-search-transport.docs.staging.corp.mongodb.com/v2/status

shows request being made to staging search server:

image

Copy link

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

Dockerfile.enhanced Outdated Show resolved Hide resolved
@seungpark seungpark marked this pull request as ready for review November 21, 2023 19:05
Copy link
Contributor

@mmeigs mmeigs left a comment

Choose a reason for hiding this comment

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

Looks good! Only thing is that I believe preprd is actually dotcomstg not stg. stg is content writers staging.

Dockerfile.enhanced Outdated Show resolved Hide resolved
@seungpark
Copy link
Contributor Author

Looks good! Only thing is that I believe preprd is actually dotcomstg not stg. stg is content writers staging.

yes! i updated both the param store values for dotcomstg and stg (stg was for testing purposes!) should see the changes when we deploy docs-qa as well 👍

@seungpark seungpark merged commit bbc701f into master Nov 27, 2023
7 checks passed
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.

2 participants