Skip to content
This repository has been archived by the owner on Jul 29, 2019. It is now read-only.

Use textContent over innerHTML so you can bind untrusted text values #7

Merged
merged 1 commit into from
Feb 10, 2019

Conversation

codymikol
Copy link
Contributor

Using innerHTML allows for XSS attacks, textContent does not allow this.

@bersLucas
Copy link
Owner

Can you run yarn run default and commit the results so the docs are updated too?

@codymikol
Copy link
Contributor Author

codymikol commented Feb 10, 2019

Sure, but going forward I'd recommend checking out the the docs for npm scripts, essentially you can hook all of the building and cleaning up of your dist so it doesn't have to be maintained in the repo.

https://docs.npmjs.com/misc/scripts

tl;dr; running npm publish will create all of that stuff, publish it, and clean it out so you don't have to bother committing all of it every time.

🐔

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

Successfully merging this pull request may close these issues.

2 participants