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

Fix acceptance test failures caused by HostAuthorization in Sinatra 4.10 #165

Merged
merged 2 commits into from
Dec 17, 2024

Conversation

koetsier
Copy link
Contributor

The acceptance tests failed because Sinatra 4.1.0 introduced HostAuthorization sinatra/sinatra#2053, which rejects connections from remote hosts by default unless explicitly allowed. This behavior affects environments where the app is accessed from remote addresses during tests.

Setting APP_ENV to production fixes that, allowing the tests to pass without further configuration. Updated the environment setup to include this fix.

Link to Jira card (if applicable):
https://technologyprogramme.atlassian.net/browse/GW-2053

….1.0

The acceptance tests failed because Sinatra 4.1.0 introduced `HostAuthorization` sinatra/sinatra#2053, which rejects connections from remote hosts by default unless explicitly allowed. This behavior affects environments where the app is accessed from remote addresses during tests.

Setting `APP_ENV` to `production` fixes that, allowing the tests to pass without further configuration. Updated the environment setup to include this fix.
Updated to use `docker compose`, the newer and recommended way of managing multi-container Docker applications, replacing the deprecated `docker-compose` command.
Copy link

@szd55gds szd55gds left a comment

Choose a reason for hiding this comment

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

LGTM, merge when ready

@koetsier koetsier merged commit d774ec8 into master Dec 17, 2024
1 check 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