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

Default block mapstore proxy #421

Merged
merged 2 commits into from
Sep 4, 2024
Merged

Default block mapstore proxy #421

merged 2 commits into from
Sep 4, 2024

Conversation

f-necas
Copy link
Contributor

@f-necas f-necas commented Sep 4, 2024

Because mapstore proxy is open to every domain unless

@f-necas f-necas merged commit f9d431a into master Sep 4, 2024
@f-necas f-necas deleted the mapstore-default-block branch September 4, 2024 13:33
Copy link

github-actions bot commented Sep 4, 2024

💔 All backports failed

Status Branch Result
docker-master An unhandled error occurred. Please see the logs for details
24.0 An unhandled error occurred. Please see the logs for details
docker-24.0 An unhandled error occurred. Please see the logs for details

Manual backport

To create the backport manually run:

backport --pr 421

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

f-necas added a commit that referenced this pull request Sep 4, 2024
f-necas added a commit that referenced this pull request Sep 4, 2024
f-necas added a commit that referenced this pull request Sep 4, 2024
@fvanderbiest
Copy link
Member

Unfortunate you did not take advantage of the PR to update the comment, which is misleading, at the same time...

# hostname whitelist limit the accepted requesting hosts (where the client application is hosted), to avoid the proxy being used by anyone you should 
# set the list to accept only the host(s) where mapstore is deployed

landryb added a commit to georchestra/ansible that referenced this pull request Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants