Bug 212049 - textproc/py-sphinx: [PATCH] Enable loading of object inventories via https
Summary: textproc/py-sphinx: [PATCH] Enable loading of object inventories via https
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Kris Moore
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-08-22 14:13 UTC by Melvyn Sopacua
Modified: 2016-09-20 14:44 UTC (History)
0 users

See Also:
m.r.sopacua: maintainer-feedback-


Attachments
Add CA_BUNDLE option (770 bytes, patch)
2016-08-22 14:13 UTC, Melvyn Sopacua
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Melvyn Sopacua 2016-08-22 14:13:54 UTC
Created attachment 173938 [details]
Add CA_BUNDLE option

Loading the python object inventory uses https. Without the CA bundle installed, this fails via the standard urllib2 error [1].

Patch also bumps PORTREVISION, which may or may not be what maintainer wants.

[1] https://www.quora.com/Are-your-Python-scripts-failing-like-this-urllib2-URLError-urlopen-error-SSL-CERTIFICATE_VERIFY_FAILED-certificate-verify-failed
Comment 1 Melvyn Sopacua 2016-09-09 12:59:34 UTC
I've noticed several object inventories have moved to https. Django, among others:

curl -D /dev/stdout -o /dev/null --silent http://docs.djangoproject.com/en/dev/_objects/
HTTP/1.1 301 Moved Permanently
Server: nginx
Date: Fri, 09 Sep 2016 12:53:08 GMT
Content-Type: text/html
Content-Length: 178
Connection: keep-alive
Location: https://docs.djangoproject.com/en/dev/_objects/
Comment 2 commit-hook freebsd_committer 2016-09-20 14:38:02 UTC
A commit references this bug:

Author: kmoore
Date: Tue Sep 20 14:37:12 UTC 2016
New revision: 422502
URL: https://svnweb.freebsd.org/changeset/ports/422502

Log:
  - Add RUN_DEPEND on security/ca_root_nss
  - Bump PORTREV

  PR: 212049

Changes:
  head/textproc/py-sphinx/Makefile
Comment 3 Kris Moore freebsd_committer 2016-09-20 14:44:18 UTC
I've just made it a default depend. Thanks!