Skip to content

Commit

Permalink
bug symfony#36518 [Security] Fix missing nullable in CsrfTokenBadge (…
Browse files Browse the repository at this point in the history
…sstok)

This PR was merged into the 5.1-dev branch.

Discussion
----------

[Security] Fix missing nullable in CsrfTokenBadge

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       |
| License       | MIT
| Doc PR        |

Related to symfony#33558 I noticed a minor 🤏  bug with the method return-type.

Commits
-------

5cb633c Update CsrfTokenBadge.php
  • Loading branch information
chalasr committed Apr 21, 2020
2 parents e36ea50 + 5cb633c commit 6b682bf
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -45,7 +45,7 @@ public function getCsrfTokenId(): string
return $this->csrfTokenId;
}

public function getCsrfToken(): string
public function getCsrfToken(): ?string
{
return $this->csrfToken;
}
Expand Down

0 comments on commit 6b682bf

Please sign in to comment.