Skip to content
This repository has been archived by the owner on Dec 2, 2021. It is now read-only.

Review Grommet UI React Library #108

Open
dscrobonia opened this issue Apr 29, 2020 · 2 comments
Open

Review Grommet UI React Library #108

dscrobonia opened this issue Apr 29, 2020 · 2 comments
Assignees
@dscrobonia
Labels
Security Pair with priority label to help communicate risk/urgency

Comments

@dscrobonia
Copy link

Namely look to make sure Markdown component is safe
@dscrobonia dscrobonia self-assigned this Apr 30, 2020
@dscrobonia dscrobonia added the Security Pair with priority label to help communicate risk/urgency label Apr 30, 2020
@dscrobonia
Copy link
Author

grommet UI Markdown component uses markdown-to-jsx which appears to be dangerouslySetInnerHtml free which is 👍

grommetUI also not using dangerouslySetInnerHtml

@dscrobonia
Copy link
Author

looks fine to me.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@dscrobonia dscrobonia

Labels
Security Pair with priority label to help communicate risk/urgency
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dscrobonia