We already escape attribute values in this fashion. There still exists a possible attack vector, which is visible as a failed test in the current version of the code.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
We already escape attribute values in this fashion. There still exists a possible attack vector, which is visible as a failed test in the current version of the code.