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.

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

Prevent tag injection #20

Open
eliemichel opened this issue Jul 30, 2014 · 3 comments
Open

Prevent tag injection #20

eliemichel opened this issue Jul 30, 2014 · 3 comments

Comments

@eliemichel
Copy link
Member

When importing tags directly from feed, some of them can be system tags or even virtual tags. We have to be aware of that, although I don't exactly know what behavior to expect.

My idea is that:

  • System tags may be leaved un escaped, because it could be interesting for some advanced features such as recognizing a particular entry format and so display it in another way. But it would be anoying for tags such as _sticky or _no_home. We could either blacklist or on the contrary whitelist some tags.
  • Virtual tags better be escaped. Maybe by a html entity that would enable a pretty printing rom the user point of view.
@Phyks Phyks added easytask and removed discussion labels Aug 7, 2014
@Phyks Phyks changed the title Prevent tag injection [EasyTask] Prevent tag injection Aug 7, 2014
@Phyks Phyks added discussion and removed bug labels Aug 8, 2014
@Phyks Phyks changed the title [EasyTask] Prevent tag injection Prevent tag injection Aug 8, 2014
@Phyks
Copy link
Member

Phyks commented Sep 29, 2014

Up. Any new ideas on this topic?

@eliemichel
Copy link
Member Author

Is it really an urgent feature?
Actually we did not decide of the expected behavior…

@Phyks
Copy link
Member

Phyks commented Sep 29, 2014

Not urgent. Just to know wether someone had clearer ideas about this subject or not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants