Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix cookie strict #756

Merged
merged 5 commits into from
Aug 5, 2024
Merged

Fix cookie strict #756

merged 5 commits into from
Aug 5, 2024

Conversation

jbrown-xentity
Copy link
Contributor

Add necessary config items from ckan2.10 changelog: https://docs.ckan.org/en/2.10/changelog.html#migration-notes-2-10

Pull Request

Related to GSA/data.gov#2138

About

Comes from https://docs.ckan.org/en/2.10/changelog.html#migration-notes-2-10

PR TASKS

  • The actual code changes.
  • Tests written and passed.
  • Any changes to docs?
  • Any new
    requirements versions
    to pull in?

Change to lax
@jbrown-xentity
Copy link
Contributor Author

This has been tested in dev (see PR). This is ready for review and release.

@jbrown-xentity jbrown-xentity requested a review from a team August 1, 2024 21:53
@rshewitt
Copy link
Contributor

rshewitt commented Aug 5, 2024

dev ckan token
Screenshot 2024-08-05 at 1 16 10 PM

@jbrown-xentity jbrown-xentity merged commit 6131fff into main Aug 5, 2024
6 checks passed
@jbrown-xentity jbrown-xentity deleted the feature/secure-cookie branch August 5, 2024 20:17
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