Skip to content
This repository has been archived by the owner on Feb 22, 2024. It is now read-only.

- Log ConnectionError exceptions #196

Open
wants to merge 15 commits into
base: future
Choose a base branch
from
Open

- Log ConnectionError exceptions #196

wants to merge 15 commits into from

Conversation

jorik90
Copy link

@jorik90 jorik90 commented Jan 26, 2016

  • verify=False for connections to OneDrive api
  • Ignore InsecureRequestWarning
  • Fix logging to file

@lamarios
Copy link

Does this require a specific python module ? I tried to run your branch and had the following error:
ImportError: cannot import name 'InsecureRequestWarning'

@jorik90
Copy link
Author

jorik90 commented Feb 15, 2016

Weird. I'm just running a pretty clean Raspbian (Debian) installation. Could you try commenting out line 31 and 100 in onedrive_d/od_onedrive_api.py? It should be in the urllib3 package, which is required anyway.

@lamarios
Copy link

It seems to work this way. Synchronization is going on pretty well.

@jorik90
Copy link
Author

jorik90 commented Feb 17, 2016

Weird. I guess you have tons of warnings in the log file? That's the reason I added it.

There's only 1 issue I encouter personally about a few dozen times. Sometimes de API does not return the correct file size. This is giving me lot's of duplicates, mostly for PDF's and some photos. Problem is described here: http://stackoverflow.com/a/27031491. I'll try to work around that as well.

I'll also check if I can only filter out the InsecureRequestWarning's when the import is available.

Jorik Berkepas and others added 4 commits February 17, 2016 14:35
Requests 2.11.0 documentation: Some previous releases accidentally accepted non-strings as acceptable header values. This release does not.

This solved the error:
requests.exceptions.InvalidHeader: Header value 961 must be of type str or bytes, not <class 'int'>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants