-
Notifications
You must be signed in to change notification settings - Fork 68
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Type check the empty string when creating repsonse
When creating a response we type check that the user has passed the correct type to content and text, however we check this based on whether the value is True and not whether it has been passed at all which means that the empty string of the wrong type can incorrectly pass through this check. Fix this check to test the empty string. Anyone relying on this passing will be caught with a more confusing TypeError later and so this should be backwards compatible. Change-Id: I826da9b7fd83bb88af50e4a96a5e6358ee35e4b2 Closes-Bug: #1647880
- Loading branch information
1 parent
95dffef
commit 86fb33d
Showing
3 changed files
with
28 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters