Skip to content

Nginx Config Update#58

Merged
greenbonebot merged 2 commits intomainfrom
nginx-config
Feb 25, 2026
Merged

Nginx Config Update#58
greenbonebot merged 2 commits intomainfrom
nginx-config

Conversation

@bjoernricks
Copy link
Contributor

@bjoernricks bjoernricks commented Feb 25, 2026

What

  • Increase client max body upload size
  • Return 404 for GSA config files

Why

  • Report Uploads may be huge files
  • Don't return index.html for the config.js file if it is missing to avoid parsing errors in the browser

@bjoernricks
Change: Allow to upload large files to gsad
551c52e 
XML Reports to be imported may be very large files. Therefore the client
max body size needs to be increased. Use same size as in GOS.
@bjoernricks
Change: Return 404 if config.js is not available
2a4b273 
Instead of returning the index.html file for the config.js file return a
404.
@bjoernricks bjoernricks requested review from a team as code owners February 25, 2026 08:08
@greenbonebot greenbonebot enabled auto-merge (rebase) February 25, 2026 08:08
@greenbonebot greenbonebot merged commit ceae6d2 into main Feb 25, 2026
35 checks passed
@greenbonebot greenbonebot deleted the nginx-config branch February 25, 2026 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ozgen ozgen ozgen approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bjoernricks @ozgen @greenbonebot