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.

Sign up for GitHub

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

Jump to bottom

BufferedHtmlContent must encode strings which are appended. #34

Closed
sornaks opened this issue Aug 12, 2015 · 5 comments
Closed

BufferedHtmlContent must encode strings which are appended. #34

sornaks opened this issue Aug 12, 2015 · 5 comments
Assignees
@rynowak
Milestone
1.0.0-beta8

Comments

@sornaks
Copy link

sornaks commented Aug 12, 2015

Currently the strings which are appended using BufferedHtmlContent.Append(string) are not encoded when written. We must encode them. #32 (comment)
@dougbu
Copy link
Member

dougbu commented Aug 12, 2015

Could we remove MVC's StringHtmlContent when reacting to a fix for this issue in that repo?

@rynowak
Copy link
Member

rynowak commented Aug 12, 2015

👍

@danroth27 danroth27 added this to the 1.0.0-beta8 milestone Aug 18, 2015
@danroth27 danroth27 added the task label Aug 18, 2015
@Eilon Eilon assigned dougbu and unassigned sornaks Aug 31, 2015
@rynowak rynowak assigned rynowak and unassigned dougbu Sep 11, 2015
@rynowak
Copy link
Member

rynowak commented Sep 11, 2015

Poached!

@rynowak rynowak mentioned this issue Sep 11, 2015
Closed
@dougbu
Copy link
Member

dougbu commented Sep 11, 2015

👍

@rynowak
Copy link
Member

rynowak commented Sep 16, 2015

8376b8d

@rynowak rynowak closed this as completed Sep 16, 2015
@ghost ghost locked as resolved and limited conversation to collaborators Dec 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rynowak rynowak

Labels
None yet
Projects
None yet
Milestone
1.0.0-beta8
Development

No branches or pull requests
4 participants
@rynowak @danroth27 @dougbu @sornaks