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

Loading comments over HTTP instead of HTTPS #8

Closed
emitchel opened this issue Jan 28, 2016 · 6 comments
Closed

Loading comments over HTTP instead of HTTPS #8

emitchel opened this issue Jan 28, 2016 · 6 comments

Comments

@emitchel
Copy link

Error retrieving comments over HTTP instead of HTTPS

@asadw1
Copy link

asadw1 commented Jan 28, 2016

I experienced similar issues. Was it error fetching data?
On Jan 28, 2016 7:45 AM, "Elliot Mitchell" notifications@github.com wrote:

Error retrieving comments over HTTP instead of HTTPS


Reply to this email directly or view it on GitHub
#8.

@emitchel
Copy link
Author

image

Seems to be a simple fix

@jasonbio
Copy link
Owner

Weird, it should request everything over HTTPS by default.. wondering if maybe reddit was temporarily downgrading requests to HTTP. Can't reproduce right now anyways.

@alminisl
Copy link

alminisl commented Apr 7, 2016

I get the same issue when I want to load some Posts.

@jasonbio
Copy link
Owner

jasonbio commented Apr 7, 2016

Can you please provide me with some console logs and network requests? Also what commands were you typing? I can't seem to replicate this myself so I'm having trouble figuring out what the problem is.

My only theory is that the "view comments" command is requesting "https://reddit.com/r/subreddit/comments/4dthkd/etc..." instead of "https://www.reddit.com/r/subreddit/comments/4dthkd/etc..." (missing the www) which seems to result in a 301 Moved Permanently response in certain cases and then forwards to "https://www.reddit.com/r/subreddit/comments/4dthkd/etc...". Not sure why the request would be downgraded after the 301 response, but it's possible I guess.

@jasonbio
Copy link
Owner

jasonbio commented Apr 8, 2016

I addressed this issue in my latest commit - hopefully this helps 4323c5e

@jasonbio jasonbio closed this as completed Apr 8, 2016
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

No branches or pull requests

4 participants