Skip to content

Update Compose File#617

Draft
bjoernricks wants to merge 3 commits intomainfrom
compose-file
Draft

Update Compose File#617
bjoernricks wants to merge 3 commits intomainfrom
compose-file

Conversation

@bjoernricks
Copy link
Contributor

@bjoernricks bjoernricks commented Feb 27, 2026

What

Update Compose File for change nginx generation

Why

Despite not being able to reproduce the host not found in upstream error the nginx config generation is changed in the gvm-config container. This PR adapts the compose file to the new config file generation.

References

Closes #616

The container now renders/creates full nginx config files and not nginx
templates anymore. This also has the advantage we can use sane defaults
which don't need to be set as environment variables.
Adapt opening a browser window to new default settings.
@bjoernricks bjoernricks requested a review from a team as a code owner February 27, 2026 13:13
@bjoernricks bjoernricks marked this pull request as draft February 27, 2026 13:13
@greenbonebot greenbonebot enabled auto-merge (rebase) February 27, 2026 13:13
@github-actions
Copy link

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 09a3f3a.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

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.

Nginx doesn't start

1 participant