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

Added line to give more descriptive error #24336

Merged
merged 1 commit into from Jun 4, 2015
Merged

Added line to give more descriptive error #24336

merged 1 commit into from Jun 4, 2015

Conversation

twangboy
Copy link
Contributor

@twangboy twangboy commented Jun 2, 2015

Problem when trying to download https from docs.saltstack.com since the new move to wpengine. The cp.get_url would just stacktrace. Now it includes a reason why.
Fixes: #24154

@jfindlay jfindlay added Minor Change Platform Relates to OS, containers, platform-based utilities like FS, system based apps File Servers labels Jun 2, 2015
justinta pushed a commit that referenced this pull request Jun 4, 2015
Added line to give more descriptive error
@justinta justinta merged commit 485116c into saltstack:2015.5 Jun 4, 2015
@twangboy twangboy deleted the fix_cp_get_url branch February 18, 2016 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
File Servers Platform Relates to OS, containers, platform-based utilities like FS, system based apps Tests-Passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants