uv icon indicating copy to clipboard operation
uv copied to clipboard

Published 20 hours ago • verified publisher icon astral-sh

Add unit test coverage for netrc authentication

Open zanieb opened this issue 1 year ago • 2 comments

Similar to #2447 — but we should use a netrc file instead of in-url authentication.

We'll probably need to configure netrc to use a file in a temporary directory. We don't want this to affect the developer's machine.

zanieb avatar Mar 14 '24 18:03 zanieb

Is this test not sufficient to validate netrc authentication? Or do we need more tests to validate correct precedence?

bschoenmaeckers avatar Mar 14 '24 22:03 bschoenmaeckers

It's not sufficient because it does doesn't ensure we're properly applying authentication to all the requests necessary for installation. I'm looking for something that's end-to-end like https://github.com/astral-sh/uv/pull/2463

zanieb avatar Mar 14 '24 23:03 zanieb

We have more test coverage in https://github.com/astral-sh/uv/pull/2976 now but still not end-to-end.

zanieb avatar Apr 16 '24 16:04 zanieb