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

bugfix svg loader for invalid / missing mimetype #1942

Merged
merged 1 commit into from
Dec 18, 2018
Merged

bugfix svg loader for invalid / missing mimetype #1942

merged 1 commit into from
Dec 18, 2018

Conversation

GPlay97
Copy link
Contributor

@GPlay97 GPlay97 commented Oct 3, 2018

If invalid svg or svg without valid mimetype was given, the module crashed when verifiying the svg file in the isSVG method.

I have fixed this with a little check to ensure that mimetype is valid, before checking if it's an svg.

@Samuell1 Samuell1 merged commit 4fa84a4 into vuematerial:dev Dec 18, 2018
marcosmoura added a commit that referenced this pull request May 11, 2019
* origin: (55 commits)
  fix(MdSvgLoader): svg loader for invalid / missing mimetype (#1942)
  docs(Icon): fix minor typo (#1973)
  fix(MdRouterLink): new router-link registration name (#1978)
  chore: apply discord vanity URL (#1927)
  docs(ROADMAP.md): replace dead slack invite link with discord invite link (#1924)
  docs(Table): fix grammar issues (#1902)
  feat: router link components improvements (#1651)
  fix(MdTable): recalculate fixed header width on data changes (#1877)
  fix(MdDialogContent): missing theme class (#1876)
  docs(Elevation and Layout): Typos fixed and improved documentation. (#1878)
  fix(MdTable): nextTick selection data (#1874)
  docs(BottomBar): fix typo (#1875)
  docs(MdButton): escape a tag (#1872)
  docs(Input): typos and correction
  chore: upgrade dependencies
  fix(MdField): remove firefox required box-shadow
  feat: disable complexity rules for some methods
  fix(MdProgressSpinner): fix CSP error (#1850)
  fix: MdBadge do not accept color #1854 issue (#1856)
  fix: fix the repeated generated css theme classes if there is multiple themes defined (#1784)
  ...
marcosmoura added a commit that referenced this pull request May 12, 2019
* origin: (80 commits)
  docs: remove wrong text-shadow
  docs: add referal code
  fix(MdProgressSpinner): improve transition (#1946)
  chore: upgrade deps
  docs: fix 2 wrong property names with autocomplete docs (#2006)
  fix(MdTable): undefined table sorting #1906 (#2012)
  docs: fix dynamic tooltip text to show bottom instead of top (#2013)
  refactor(MdDatepicker): upgrade date-fns (#2037)
  fix(MdDatepicker): positioning when used inside flex container (#2036)
  fix(MdSelect): infinite loop error, when no v-model provided (#2035)
  fix(MdApp): normalized component's tag before checking to match slot … (#1994)
  chore: some misspellings in config.js (#1986)
  fixFixed non unique ids on MdSelect (#2001)
  fix(MdInput): invert password on-off icons (#2008)
  test: fix test selectors for checkbox and switch (#2009)
  docs: add default theme as a suggestion (#2043)
  Fixed md-wave transition (#2034)
  fix(MdRipple): fixed missing ripple effect (#2059)
  docs: add codefund ads and remove old sponsors
  fix(MdSvgLoader): svg loader for invalid / missing mimetype (#1942)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants