Skip to content

+netrc for authentication#85

Open
shamilbi wants to merge 1 commit intoido50:mainfrom
shamilbi:netrc
Open

+netrc for authentication#85
shamilbi wants to merge 1 commit intoido50:mainfrom
shamilbi:netrc

Conversation

@shamilbi
Copy link
Copy Markdown
Collaborator

@shamilbi shamilbi commented Feb 27, 2026 •
edited
Loading

i hope it closes #63, could not test it against a real index that requires authentication

@grische
Copy link
Copy Markdown
Contributor

grische commented Mar 3, 2026

i hope it closes #63, could not test it against a real index that requires authentication

I am against this kind of large code change if it cannot be tested at all. This probably conflicts heavily with #82 and I'd rather have this code here follow a similar pattern regarding the class structure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Special credential characters in extra index url

2 participants

@shamilbi @grische