Skip to content

Comments

Maintenance: Hint CSRF error fix#767

Merged
ralf401 merged 1 commit intozammad:pre-releasefrom
dargmuesli:patch-1
Feb 23, 2026
Merged

Maintenance: Hint CSRF error fix#767
ralf401 merged 1 commit intozammad:pre-releasefrom
dargmuesli:patch-1

Conversation

@dargmuesli
Copy link
Contributor

@MrGeneration
Copy link
Member

In my opinion this is indirectly already available:

https://docs.zammad.org/en/latest/getting-started/configure-webserver.html?highlight=csrf

image

@dargmuesli
Copy link
Contributor Author

I see, interesting find. I've primarily been looking at the docker documentation for my setup, that's why I think it's reasonable to add a hint to it too. The webserver setup page reads to be aimed at people who actually want to go into configuring a webserver which you'd like to generally "avoid" in the Docker case in which environment variables are the configuration api in that sense. Thus, I think a pointer to the potential CSRF issue fix in the environment variable documentation is very helpful.


Ah, speaking about which, I just notice that the page I wanted to edit is this:

But on the current default branch that seems to have been moved to:

But I think the reason given above still stands.

@ralf401
Copy link
Contributor

ralf401 commented Feb 23, 2026

@dargmuesli thanks for your PR. Looks good to me and I would argue it doesn't hurt to have it there too.

@ralf401 ralf401 merged commit 90b88c9 into zammad:pre-release Feb 23, 2026
1 check passed
@dargmuesli dargmuesli deleted the patch-1 branch February 23, 2026 16:23
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.

3 participants