Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

Use ckan.ini rather than production.ini #27

Closed
wants to merge 1 commit into from

Conversation

kentsanggds
Copy link
Collaborator

What

This PR depends on alphagov/ckanext-datagovuk#1383 to be merged into main-2.10 as the config name will be changed, and the config file has been updated.

Reference

https://trello.com/c/bEsbEk1m/1151-run-security-check-and-resolve-critical-issues

## CSRF Protection #############################################################
WTF_CSRF_ENABLED = true
WTF_CSRF_CHECK_DEFAULT = true
WTF_CSRF_SECRET_KEY = _e6qkoYldyll9U3zfy5n6rg-JxCPY4DUfy5LHu3qnUo
Copy link

Choose a reason for hiding this comment

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

If this is actually used then you'll probably need to change it now 😅 If it's a dummy value or unused, then it's a good idea to change it to something that's more obviously a dummy/placeholder value so that it's clear that it's ok.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll probably auto generate this in the init

@kentsanggds
Copy link
Collaborator Author

Superceded by other PRs

@kentsanggds kentsanggds closed this Oct 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants