Skip to content

Commit a8c2b8e

Browse files
konstinGankra
authored andcommitted
Stabilize uv publish (#11032)
`uv publish` has not changed for some time, it has [notable production usage](https://github.com/search?q=%22uv+publish%22&type=code) and there are no outstanding blockers, it is time to stabilize it with the 0.6 release. ## Introduction Publishing is only usable through `uv publish`. You need to build source distributions and wheels ahead of time, usually with `uv build`. By default, `uv publish` will upload all source distributions and wheels in the `dist/` folder, ignoring all non-matching filenames. By default, `uv build` and most other build frontend write their artifacts to `dist/`. Together, we can build a publish workflow including a smoke test that all relevant files have actually been included in the wheel: ``` uv build uv venv uv pip install --find-links dist ... uv run smoke_test.py uv publish ``` ## Project configuration There are 3 options supported in configuration files: - `tool.uv.publish-url` - `tool.uv.trusted-publishing` - `tool.uv.check-url` ## Index configuration Options support on the CLI and through environment variables for index configuration: ``` --index <INDEX> The name of an index in the configuration to use for publishing [env: UV_PUBLISH_INDEX=] --publish-url <PUBLISH_URL> The URL of the upload endpoint (not the index URL) [env: UV_PUBLISH_URL=] --check-url <CHECK_URL> Check an index URL for existing files to skip duplicate uploads [env: UV_PUBLISH_CHECK_URL=] ``` There are two ways to configure `uv publish`: Passing options individually or using the index API. For the individual options, there `--publish-url` and `--check-url`, and their configuration counterparts, `tool.uv.publish_url` and `tool.uv.check_url`. `--publish-url` is named this way to be clearly different from the simple index URL, since uploading to the index URL leads to unclear errors, or worse a 200 OK with no effect. While we intend to keep supporting this configuration, the index API is better integrated. In the index API, the user specifies `[[tool.uv.index]]`, with an index name, the simple index URL and the publish URL. The `publish-url` and `url` are equivalent to `--publish-url` and `--check-url`. The `url` being mandatory makes for a better upload behavior (next paragraph). ```toml [[tool.uv.index]] name = "pypi" url = "https://pypi.org/simple" publish-url = "https://upload.pypi.org/legacy/" ``` ## Existing files and API limitations A version of a package contains multiple files, for pure-python packages usually a source distribution and a wheel, for native packages usually many, larger wheels and a source distributions. Uploads in the not officially specified Upload API 1.0 are file based: Once you upload a file, the version is created, even though most files are still missing. When uploading a series of files fails in the middle (e.g. the CI server breaks), the release is only half uploaded. For such cases, you want to re-try the upload. The response of an index when re-uploading a file is implementation defined. Notably, PyPI accepts uploads of the same file again with status 200, but rejects uploads of a file with the same name but different contents with status 400. Other indexes reject all attempts at re-uploads with different status codes and messages. Twine handles this with `--skip-existing`, which allows ignoring errors due to files with the same name as an existing file being uploaded, however this does also not error when uploading a file with different contents but the same name, which indicates a problem with the publish pipeline. To properly solve this, we need the ability to stage releases: Files of a version are uploaded to a staging area, and only when all files are uploaded, we atomically publish the release. When an upload breaks or CI fails, we can discard or overwrite the staging area and try again. This will only be properly solved by PEP 694 "Upload 2.0 API for Python Package Indexes", with unclear progress. For local publishing, it would also be convenient to be able to check which files exist and what their hashes are from only the publish URL, so files in the `dist/` folder from a previous release can be ignored. In the Upload API 1.0, we need to upload transformed METADATA fields along with the file as form-data. We currently upload only recognized metadata fields, where we know how to translate the field name to the form-data name. This means when a user adds unknown, wrong or future-PEP metadata we miss it. To me best knowledge no index currently verifies that the form-data and the METADATA file in the wheel match. Upload API 2.0 will be an entirely new protocol. It is unclear how we will decide whether to use Upload API 1.0 or Upload API 2.0 once the latter is released. Upload API 2.0 will remove the need for a check URL. This means no changes for `--index`, but `--check-url` will be incompatible with Upload API 2.0. ## Authentication Options support on the CLI and through environment variables for authentication: ``` -u, --username <USERNAME> The username for the upload [env: UV_PUBLISH_USERNAME=] -p, --password <PASSWORD> The password for the upload [env: UV_PUBLISH_PASSWORD=] -t, --token <TOKEN> The token for the upload [env: UV_PUBLISH_TOKEN=] --trusted-publishing <TRUSTED_PUBLISHING> Configure using trusted publishing through GitHub Actions [possible values: automatic, always, never] --keyring-provider <KEYRING_PROVIDER> Attempt to use `keyring` for authentication for remote requirements files [env: UV_KEYRING_PROVIDER=] [possible values: disabled, subprocess] ``` We need credentials for the publish URL, and we may need credentials for the check URL. We support credentials from environment variables, the CLI, the URL, the keyring, trusted publishing or a prompt. The username can come from, in order: - Mutually exclusive: - `--username` or `UV_PUBLISH_USERNAME`. The CLI option overrides the environment variable - The username field in the publish URL - If `--token` or `UV_PUBLISH_TOKEN` are used, it is `__token__`. The CLI option overrides the environment variable - If trusted publishing is available, it is `__token__` - (We currently do not read the username from the keyring) - If stderr is a tty, prompt the user The password can come from, in order: - Mutually exclusive: - `--password` or `UV_PUBLISH_PASSWORD`. The CLI option overrides the environment variable - The password field in the publish URL - If `--token` or `UV_PUBLISH_TOKEN` are used, it is the token value. The CLI option overrides the environment variable - If the keyring is enabled, the keyring entry for the URL and username - If trusted publishing is available, the trusted publishing token - If stderr is a tty, prompt the user If no credentials are found, we do a final check in the auth middleware cache and otherwise error without sending the request. Trusted publishing is only supported in GitHub Actions. By default, we try to retrieve a token from it in GitHub Actions (`GITHUB_ACTIONS` is `true`) but continue even it this fails. Trusted publishing can be forced with `--trusted-publishing always`, to error on misconfiguration, or deactivated with `--trusted-publishing never`. The option can also be configured through `tool.uv.trusted-publishing`. When `--check-url` or `--index` are used, we may need credentials for the index URL, too. These are handle separately by the same rules as using the index anywhere else. The `--keyring-provier` option is however shared between them, turning the keyring on for either turns it on for both. As future option, we could read `UV_INDEX_USERNAME` and `UV_INDEX_PASSWORD` as fallbacks for the publish credentials (#9845). This however would clash with prompting: When index credentials and upload credentials are not the same (they usually should be different, since regular uv operations should have less privileges than publish), we would then instead of prompting use the wrong credentials from `UV_INDEX_*` and fail. A major UX problem is that there is no standard for the username when using a token (or rather, there is no standard for just sending a token without a username). PyPI uses `__token__`, Cloudsmith used to use your username or `token`, but now also supports `__token__` (#8221), while Google Cloud Artifacts always uses `oauth2accesstoken` (#9778). This means the index documentation may say you're getting a token for authentication, but you must not use `--token`, you must instead set username and password. This is something that we can hopefully fix with Upload API 2.0. An unsolved problem with the keyring is that you it's best practice to use publish tokens scoped to projects and store tokens in a secure location such as the keyring, but the keyring saves a single password per publish URL and username combination. That means that it can't natively store separate passwords for publishing multiple packages. The current hack around this is using the package name as query parameter, e.g. `https://test.pypi.org/legacy/?astral-test-keyring`, as PyPI ignores this query parameter. This is however only applicable when publishing locally and not from CI. Another problem is that the keyring implementation currently relies on the `keyring` pypi package, which needs to be installed in PATH together with its plugins and is comparatively slow. This would be improved by native keyring support (#10867), with the same caveats such as keyring plugins that shared with the simple index API. ## Missing and unsupported features We currently don't upload attestations (PEP 740). Attestations are an additional field in the form-data, so we should be able to add them transparently without any changes to the API, unless we want to add a switch to deactivate even when trusted publishing is used. See also https://trailofbits.github.io/are-we-pep740-yet/. Setuptools is writing an invalid combination of Metadata-Version and used metadata fields in some cases, which PyPI correctly rejects (#9513). We set a 15min overall timeout since reqwest is missing a write timeout option (seanmonstar/reqwest#2403). #8641 and #8774: We build artifact checking in some capacity. This should be done ideally by the build backend or at latest as part of `uv build`, doing it as part of publish is too late. Closes #7839 --- Let me know if i missed anything.
1 parent bb191d5 commit a8c2b8e

File tree

2 files changed

+0
-17
lines changed

2 files changed

+0
-17
lines changed

crates/uv/src/lib.rs

-4
Original file line numberDiff line numberDiff line change
@@ -1269,10 +1269,6 @@ async fn run(mut cli: Cli) -> Result<ExitStatus> {
12691269
Commands::Publish(args) => {
12701270
show_settings!(args);
12711271

1272-
if globals.preview.is_disabled() {
1273-
warn_user_once!("`uv publish` is experimental and may change without warning");
1274-
}
1275-
12761272
if args.skip_existing {
12771273
bail!(
12781274
"`uv publish` does not support `--skip-existing` because there is not a \

crates/uv/tests/it/publish.rs

-13
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ fn username_password_no_longer_supported() {
2323
----- stdout -----
2424
2525
----- stderr -----
26-
warning: `uv publish` is experimental and may change without warning
2726
Publishing 1 file to https://test.pypi.org/legacy/
2827
Uploading ok-1.0.0-py3-none-any.whl ([SIZE])
2928
error: Failed to publish `../../scripts/links/ok-1.0.0-py3-none-any.whl` to https://test.pypi.org/legacy/
@@ -49,7 +48,6 @@ fn invalid_token() {
4948
----- stdout -----
5049
5150
----- stderr -----
52-
warning: `uv publish` is experimental and may change without warning
5351
Publishing 1 file to https://test.pypi.org/legacy/
5452
Uploading ok-1.0.0-py3-none-any.whl ([SIZE])
5553
error: Failed to publish `../../scripts/links/ok-1.0.0-py3-none-any.whl` to https://test.pypi.org/legacy/
@@ -82,7 +80,6 @@ fn mixed_credentials() {
8280
----- stdout -----
8381
8482
----- stderr -----
85-
warning: `uv publish` is experimental and may change without warning
8683
Publishing 1 file to https://test.pypi.org/legacy/
8784
error: a username and a password are not allowed when using trusted publishing
8885
"###
@@ -109,7 +106,6 @@ fn missing_trusted_publishing_permission() {
109106
----- stdout -----
110107
111108
----- stderr -----
112-
warning: `uv publish` is experimental and may change without warning
113109
Publishing 1 file to https://test.pypi.org/legacy/
114110
error: Failed to obtain token for trusted publishing
115111
Caused by: Environment variable ACTIONS_ID_TOKEN_REQUEST_TOKEN not set, is the `id-token: write` permission missing?
@@ -136,7 +132,6 @@ fn no_credentials() {
136132
----- stdout -----
137133
138134
----- stderr -----
139-
warning: `uv publish` is experimental and may change without warning
140135
Publishing 1 file to https://test.pypi.org/legacy/
141136
Note: Neither credentials nor keyring are configured, and there was an error fetching the trusted publishing token. If you don't want to use trusted publishing, you can ignore this error, but you need to provide credentials.
142137
Trusted publishing error: Environment variable ACTIONS_ID_TOKEN_REQUEST_TOKEN not set, is the `id-token: write` permission missing?
@@ -162,7 +157,6 @@ fn skip_existing_redirect() {
162157
----- stdout -----
163158
164159
----- stderr -----
165-
warning: `uv publish` is experimental and may change without warning
166160
error: `uv publish` does not support `--skip-existing` because there is not a reliable way to identify when an upload fails due to an existing distribution. Instead, use `--check-url` to provide the URL to the simple API for your index. uv will check the index for existing distributions before attempting uploads.
167161
"###
168162
);
@@ -193,7 +187,6 @@ fn dubious_filenames() {
193187
----- stdout -----
194188
195189
----- stderr -----
196-
warning: `uv publish` is experimental and may change without warning
197190
warning: Skipping file that looks like a distribution, but is not a valid distribution filename: `[TEMP_DIR]/data.tar.gz`
198191
warning: Skipping file that looks like a distribution, but is not a valid distribution filename: `[TEMP_DIR]/not-a-wheel.whl`
199192
warning: Skipping file that looks like a distribution, but is not a valid distribution filename: `[TEMP_DIR]/not-sdist-1-2-3-asdf.zip`
@@ -239,7 +232,6 @@ fn check_keyring_behaviours() {
239232
----- stdout -----
240233
241234
----- stderr -----
242-
warning: `uv publish` is experimental and may change without warning
243235
Publishing 1 file to https://test.pypi.org/legacy/?ok
244236
Uploading ok-1.0.0-py3-none-any.whl ([SIZE])
245237
error: Failed to publish `../../scripts/links/ok-1.0.0-py3-none-any.whl` to https://test.pypi.org/legacy/?ok
@@ -264,7 +256,6 @@ fn check_keyring_behaviours() {
264256
----- stdout -----
265257
266258
----- stderr -----
267-
warning: `uv publish` is experimental and may change without warning
268259
Publishing 1 file to https://test.pypi.org/legacy/?ok
269260
warning: Using `--keyring-provider` with a password or token and no check URL has no effect
270261
Uploading ok-1.0.0-py3-none-any.whl ([SIZE])
@@ -291,7 +282,6 @@ fn check_keyring_behaviours() {
291282
----- stdout -----
292283
293284
----- stderr -----
294-
warning: `uv publish` is experimental and may change without warning
295285
Publishing 1 file to https://test.pypi.org/legacy/?ok
296286
Request for dummy@https://test.pypi.org/legacy/?ok
297287
Request for [email protected]
@@ -321,7 +311,6 @@ fn check_keyring_behaviours() {
321311
----- stdout -----
322312
323313
----- stderr -----
324-
warning: `uv publish` is experimental and may change without warning
325314
Publishing 1 file to https://test.pypi.org/legacy/?ok
326315
Request for dummy@https://test.pypi.org/legacy/?ok
327316
Uploading ok-1.0.0-py3-none-any.whl ([SIZE])
@@ -374,7 +363,6 @@ fn invalid_index() {
374363
----- stdout -----
375364
376365
----- stderr -----
377-
warning: `uv publish` is experimental and may change without warning
378366
error: Index not found: `bar`. Found indexes: `foo`, `internal`
379367
"###
380368
);
@@ -394,7 +382,6 @@ fn invalid_index() {
394382
----- stdout -----
395383
396384
----- stderr -----
397-
warning: `uv publish` is experimental and may change without warning
398385
error: Index is missing a publish URL: `foo`
399386
"###
400387
);

0 commit comments

Comments
 (0)