Skip to content

(PUP-1421) Remove flawed source validation in package provider appdmg.#2249

Merged
kylog merged 1 commit intopuppetlabs:masterfrom
webbj74:fix/master/PUP-1421
Jan 13, 2014
Merged

(PUP-1421) Remove flawed source validation in package provider appdmg.#2249
kylog merged 1 commit intopuppetlabs:masterfrom
webbj74:fix/master/PUP-1421

Conversation

@webbj74
Copy link
Contributor

@webbj74 webbj74 commented Jan 12, 2014

  • appdmg's "source" may be a URL that redirects to the actual dmg (and thus not end in .dmg)
  • actual download is renamed to "name" anyway
  • hdiutil doesn't require the dmg file to have a filename ending in .dmg

More Details: https://tickets.puppetlabs.com/browse/PUP-1421

* appdmg's "source" may be a URL that redirects to the actual dmg (and thus not end in .dmg)
* actual download is renamed to "name" anyway
* hdiutil doesn't require the dmg file to have a filename ending in .dmg
@puppetcla
Copy link

CLA signed by all contributors.

kylog pushed a commit that referenced this pull request Jan 13, 2014
(PUP-1421) Remove flawed source validation in package provider appdmg.
@kylog kylog merged commit 097e9fd into puppetlabs:master Jan 13, 2014
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.

3 participants