Skip to content

Ignore PermissionError in netrc_info() - #1104

Merged
lovelydinosaur merged 9 commits into
masterfrom
Jonney-patch-1
Jul 30, 2020
Merged

Ignore PermissionError in netrc_info()#1104
lovelydinosaur merged 9 commits into
masterfrom
Jonney-patch-1

Conversation

@lovelydinosaur

@lovelydinosaurlovelydinosaur commented Jul 30, 2020

Copy link
Copy Markdown
Contributor

Clean PR bringing in the work in #810.

(Because, build failures that I'd not quite figured out)

Jonneyand others added 9 commits February 11, 2020 05:03
I'm running uWSGI, the user is "nobody", the netrc_file should be '~/.netrc'.
But I don't know why, the location trunes out to be '/root/.netrc', so a PermissionError raised, it's better to ignore this error instead of to raise.
ignore (netrc.NetrcParseError, IOError, PermissionError)
Co-Authored-By: Stephen Brown II <Stephen.Brown2@gmail.com>
Co-Authored-By: Florimond Manca <florimond.manca@gmail.com>
@lovelydinosaur
lovelydinosaur deleted the Jonney-patch-1 branch July 30, 2020 14:34
@lovelydinosaur
lovelydinosaur restored the Jonney-patch-1 branch July 30, 2020 14:37
@lovelydinosaurlovelydinosaur added this to the v0.14 milestone Jul 30, 2020
@lovelydinosaurlovelydinosaur added the enhancement New feature or request label Jul 30, 2020
@lovelydinosaur

Copy link
Copy Markdown
ContributorAuthor

Not sure what's going on with the CI checks on #810, but let's get it in.

@lovelydinosaur
lovelydinosaur merged commit 9728d89 into masterJul 30, 2020
@lovelydinosaur
lovelydinosaur deleted the Jonney-patch-1 branch July 30, 2020 14:40
This was referenced Jul 30, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or request

Development

Successfully merging this pull request may close these issues.

1 participant

@lovelydinosaur