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

use https where appropriate #1961

Merged
merged 1 commit into from Aug 22, 2015
Merged

use https where appropriate #1961

merged 1 commit into from Aug 22, 2015

Conversation

@polyzen
Copy link
Contributor

polyzen commented Aug 21, 2015

No description provided.

@@ -801,7 +801,7 @@ def import_postpage_item(self, item, wordpress_namespace, out_folder=None, attac
content = get_text_tag(
item, '{http://purl.org/rss/1.0/modules/content/}encoded', '')
excerpt = get_text_tag(
item, '{http://wordpress.org/export/1.2/excerpt/}encoded', None)
item, '{https://wordpress.org/export/1.2/excerpt/}encoded', None)

This comment has been minimized.

Copy link
@Kwpolska

Kwpolska Aug 22, 2015

Member

Wrong.

@@ -917,7 +917,7 @@ def import_postpage_item(self, item, wordpress_namespace, out_folder=None, attac
def _extract_item_info(self, item):
"""Extract information about an item."""
# The namespace usually is something like:
# http://wordpress.org/export/1.2/
# https://wordpress.org/export/1.2/

This comment has been minimized.

Copy link
@Kwpolska

Kwpolska Aug 22, 2015

Member

Should not be.

@@ -587,7 +587,7 @@ def make_url(url):
description='',
lastBuildDate=datetime.datetime.utcnow(),
items=items,
generator='http://getnikola.com/',
generator='https://getnikola.com/',

This comment has been minimized.

Copy link
@Kwpolska

Kwpolska Aug 22, 2015

Member

There should be one more in nikola/nikola.py

@Kwpolska Kwpolska self-assigned this Aug 22, 2015
@Kwpolska Kwpolska added this to the v7.6.5 milestone Aug 22, 2015
@Kwpolska Kwpolska merged commit 98d1cc2 into getnikola:master Aug 22, 2015
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor AppVeyor build succeeded
Details
Kwpolska added a commit that referenced this pull request Aug 22, 2015
@polyzen polyzen deleted the polyzen:httpsallthethings branch Sep 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.