-
Notifications
You must be signed in to change notification settings - Fork 159
unable to publish #230
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
Comments
From @nex3 on July 13, 2017 20:13 I'm guessing this is a server-side issue, but just in case what do you see if you run with |
|
From @pq on July 13, 2017 20:46 FWIW, |
From @nex3 on July 13, 2017 20:54 dart-lang/sdk@e41c565 is much older than 1.25.0-dev.4.0. Could this have been a bug that has since been fixed? |
From @pq on July 13, 2017 21:7 Maybe. But |
From @nex3 on July 13, 2017 21:37 Between 1.25.0-dev.4.0 and 1.25.0-dev.6.0, Pub was upgraded from 0713718a83054fcc1c0a4b163e036f7c39ea4790 to 0f122625f8e607842afe156b9b23a9709e4ce39a, but I don't see any changes in that range that would affect publishing at all. Nor does the SDK include any changes to the HTTP infrastructure. All this makes me think that it must have been either a flake or a server-side change in the HTTP API. |
From @nshahan on July 18, 2017 23:41 I am seeing the same error on the SDK version: |
From @nshahan on July 18, 2017 23:46 I just uninstalled the dev version of the SDK and installed the stable version: That allowed me to publish without any problems. |
From @kevmoo on July 19, 2017 0:1 Yeah – this feels like a server bug. Would someone use the issue mover to...move it? |
/cc @isoos, @mkustermann |
I'm not aware of any server code change that would work with 1.24.2 but break with anything newer. The only recent change that was vaguely related to the upload was to extract the example file, but that should not care about whether the Unfortunately the pub client swallows any useful info from the exception: @kevmoo: could you please check the prod logs while @mkustermann is OOO? |
I believe the fix is in the client – due to a recent change in how storage handles uploads – dart-lang/pub#1666 (comment) |
Turns out this is a Cloud Storage issue – at least we think it is Reopening here – since storage is an implementation detail of our backend - http://b/63855519 |
Root cause: dart-lang/http#105 |
From @devoncarew on July 13, 2017 20:9
I was unable to publish a package w/ the most recent SDK:
This was for
Pub 1.25.0-dev.6.0
. @pq was able to publish the same package, using a1.25.0-dev.4.0
version of the sdk. The repo in question is https://github.com/dart-lang/cli_util.Copied from original issue: dart-lang/pub#1666
The text was updated successfully, but these errors were encountered: