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

[GH-167] - Update PWS_HOST with new value from REQ7051889 #168

Merged
merged 3 commits into from
Jan 16, 2025
Merged

Conversation

soufianerafik
Copy link
Contributor

@soufianerafik soufianerafik commented Jan 16, 2025

Context: During a recent live session, we encountered an issue with the PWS_HOST configuration while running locally. The following was observed:

  1. Using PWS_HOST=https://it-wseval1.s.uw.edu consistently resulted in 500 errors.
  2. Switching to PWS_HOST=https://wseval.s.uw.edu resolved the issue, and the application worked as expected.

After doing some research I found a change that took effect in EWS EVAL environments on 5/25/23. Any configurations or bookmarks still pointing to the old endpoint will no longer work.

Change Description:
This PR updates the PWS_HOST value in the settings/*.dotenv files to reflect the current configuration for the EWS EVAL environments. The old endpoint https://it-wseval1.s.uw.edu consistently caused 500 errors during local testing. The new endpoint https://wseval.s.uw.edu resolves this issue and aligns with changes implemented on 5/25/23.

For more information please refer to UW Connect's REQ7051889

Closes Github(s): GH-167

UW-Directory Pull Request checklist

  • I have run poetry run tox (Please check Side Note)
  • I have selected a semver-guidance: label for this pull request (under labels,
    to the right of the screen)

@soufianerafik soufianerafik added the semver-guidance:no-bump Use this when you don't want the version to change. label Jan 16, 2025
@soufianerafik soufianerafik self-assigned this Jan 16, 2025
@soufianerafik soufianerafik marked this pull request as ready for review January 16, 2025 21:35
Copy link
Contributor

Download and run the latest build with:

./scripts/run-development-server.sh -i gcr.io/uwit-mci-iam/husky-directory.app:pull-request-168

Copy link
Contributor

@goulter goulter left a comment

Choose a reason for hiding this comment

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

good fix. next time, lets find out how to code it so that we don't need the url and instead have a lookup that finds the url based on another value.

@soufianerafik soufianerafik linked an issue Jan 16, 2025 that may be closed by this pull request
@soufianerafik
Copy link
Contributor Author

good fix. next time, lets find out how to code it so that we don't need the url and instead have a lookup that finds the url based on another value.

Thanks for the review/approval, @goulter ! Yes, great idea :-)

@soufianerafik soufianerafik merged commit 5302814 into main Jan 16, 2025
2 checks passed
@soufianerafik soufianerafik deleted the GH-167 branch January 16, 2025 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-guidance:no-bump Use this when you don't want the version to change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update eval's PWS_HOST to Resolve 500 Errors When Running Locally
2 participants