Skip to content

Conversation

ericfortmeyer
Copy link
Contributor

The response filter and the token storage require access to the token.

BREAKING CHANGE: Inject the token into the CsrfResponseFilterMiddleware's constructor. The CsrfTokenGenerator has been removed.

Closes #80

The response filter and the token storage require access to the token.

BREAKING CHANGE: Inject the token into the `CsrfResponseFilterMiddleware`'s constructor.  The `CsrfTokenGenerator` has been removed.
@ericfortmeyer ericfortmeyer self-assigned this May 21, 2023
@ericfortmeyer ericfortmeyer merged commit d4ba25b into main May 21, 2023
@ericfortmeyer ericfortmeyer deleted the inject-token branch May 21, 2023 05:54
@coveralls
Copy link

coveralls commented Sep 19, 2024

Pull Request Test Coverage Report for Build 5036104664

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 96.471%

Totals Coverage Status
Change from base Build 5036048103: -0.1%
Covered Lines: 82
Relevant Lines: 85

💛 - Coveralls

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.

Inject the token
2 participants