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

add cors origin validation for portal server and auth server #4725

Merged
merged 4 commits into from
Jun 25, 2024

Conversation

shovanmaity
Copy link
Contributor

@shovanmaity shovanmaity commented Jun 21, 2024

This PR adds cors origin validation for portal server and auth server.

As frontend communicates with portal server and auth server using internal service fqdn and user access the portal using public url both are added in allowed list

Signed-off-by: Shovan Maity <shovan.maity@harness.io>
shovanmaity and others added 2 commits June 24, 2024 16:18
Signed-off-by: Shovan Maity <shovan.maity@harness.io>
Signed-off-by: Shovan Maity <shovan.cse91@gmail.com>
Signed-off-by: Shovan Maity <shovan.maity@harness.io>
@Saranya-jena Saranya-jena merged commit 17085b4 into litmuschaos:master Jun 25, 2024
19 checks passed
@shovanmaity shovanmaity deleted the cors-fix branch June 25, 2024 06:47
@shovanmaity shovanmaity changed the title add cors check add cors origin validation for portal server and auth server Jul 12, 2024
andoriyaprashant pushed a commit to andoriyaprashant/litmus that referenced this pull request Sep 7, 2024
* add cors check

Signed-off-by: Shovan Maity <shovan.maity@harness.io>

* add cors check custom middleware

Signed-off-by: Shovan Maity <shovan.maity@harness.io>

* change default url

Signed-off-by: Shovan Maity <shovan.maity@harness.io>

---------

Signed-off-by: Shovan Maity <shovan.maity@harness.io>
Signed-off-by: Shovan Maity <shovan.cse91@gmail.com>
Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>
andoriyaprashant pushed a commit to andoriyaprashant/litmus that referenced this pull request Sep 7, 2024
* add cors check

Signed-off-by: Shovan Maity <shovan.maity@harness.io>

* add cors check custom middleware

Signed-off-by: Shovan Maity <shovan.maity@harness.io>

* change default url

Signed-off-by: Shovan Maity <shovan.maity@harness.io>

---------

Signed-off-by: Shovan Maity <shovan.maity@harness.io>
Signed-off-by: Shovan Maity <shovan.cse91@gmail.com>
Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>
sagnik3788 pushed a commit to sagnik3788/litmus that referenced this pull request Oct 5, 2024
* add cors check

Signed-off-by: Shovan Maity <shovan.maity@harness.io>

* add cors check custom middleware

Signed-off-by: Shovan Maity <shovan.maity@harness.io>

* change default url

Signed-off-by: Shovan Maity <shovan.maity@harness.io>

---------

Signed-off-by: Shovan Maity <shovan.maity@harness.io>
Signed-off-by: Shovan Maity <shovan.cse91@gmail.com>
Signed-off-by: sagnik3788 <116512372+sagnik3788@users.noreply.github.com>
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.

6 participants