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 option to specify App Version #18

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Marcool04
Copy link

Hi there!
It seems, as I experienced today, that the x-pm-appversion header has a tendency to change rather often (with the versions… I guess). There are already three issues raised on the tracker related to this : #3, #7 and #11. Pull request #2 is an attempt at fixing this… but it's already broken before even being accepted since the app has bumped up in version since. Furthermore, #2 also changes the api version header x-pm-apiversion, but this does not seem necessary to actually get the email download to work (api version seems to be 3 right now, but even with x-pm-apiversion set to 1 download succeeds).
The approach used in the present pull request which is to make the appversion into a user supplied argument seems like a more robust one to me, since the program already asks users to copy-paste info from the headers anyway.
Let me know if there is anything looks wrong or needs fixing.
Regards,
Mark.

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

Successfully merging this pull request may close these issues.

1 participant