Skip to content

Comments

Maintenance: Hint CSRF error fix#767

Open
dargmuesli wants to merge 1 commit intozammad:pre-releasefrom
dargmuesli:patch-1
Open

Maintenance: Hint CSRF error fix#767
dargmuesli wants to merge 1 commit intozammad:pre-releasefrom
dargmuesli:patch-1

Conversation

@dargmuesli
Copy link

@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
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.

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