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

[AB#40732] feat: 🧑‍💻 introduce user-auth-proxy to support localhost development #123

Merged
merged 1 commit into from Jul 21, 2023
Merged

[AB#40732] feat: 🧑‍💻 introduce user-auth-proxy to support localhost development #123

merged 1 commit into from Jul 21, 2023

Conversation

EmperorRXF
Copy link
Member

@EmperorRXF EmperorRXF commented Jul 19, 2023 •
edited

  • introduced a yarn util:start-proxy script to start a localhost proxy that routes traffic to User Service
  • this shall help to avoid cross-domain cookie issues during local development
  • README.md is updated to reflect the usage

For Release Notes:

  • The Mosaic Frontend Samples now includes a utility script that allows you to start a localhost proxy for routing the traffic to the hosted Mosaic User Service. This will allow you to run the Mosaic Frontend Samples (or a similar custom frontend application) on your localhost, and connect it successfully with the hosted Mosaic User Service and get the full Authentication functionality working without any issues for setting cookies in different domains top-level domains.
  • See the PR for more details - [AB#40732] feat: 🧑‍💻 introduce user-auth-proxy to support localhost development #123

@EmperorRXF EmperorRXF force-pushed the add-user-auth-proxy branch 2 times, most recently from ba5448e to ce9116b Compare July 19, 2023 14:39
@EmperorRXF EmperorRXF requested a review from a team July 19, 2023 14:40
@EmperorRXF EmperorRXF enabled auto-merge (rebase) July 20, 2023 09:01
@EmperorRXF EmperorRXF merged commit e477389 into main Jul 21, 2023
3 checks passed
@EmperorRXF EmperorRXF deleted the add-user-auth-proxy branch July 21, 2023 06:47
@EmperorRXF EmperorRXF added the release notes The PR contains some descriptions that shall be added to the Release Notes label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@axinom-nrathnayaka axinom-nrathnayaka axinom-nrathnayaka approved these changes

Assignees
No one assigned
Labels
release notes The PR contains some descriptions that shall be added to the Release Notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@EmperorRXF @axinom-nrathnayaka