Skip to content

Remove server option from pub publish page #2612

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

Closed
santitigaga opened this issue Sep 1, 2020 · 2 comments · Fixed by #2630
Closed

Remove server option from pub publish page #2612

santitigaga opened this issue Sep 1, 2020 · 2 comments · Fixed by #2630
Labels
co.beginner Community opportunity to make a small change to documentation co.help-wanted Community opportunity to help documentation e0-minutes Can complete in < 60 minutes of normal, not dedicated, work fix.bug Needs fix of incorrect copy, code, or visual p2-medium Necessary but not urgent concern. Resolve when possible. t.cli-tools Relates to the dart command line tools

Comments

@santitigaga
Copy link

santitigaga commented Sep 1, 2020

Page URL: https://dart.dev/tools/pub/cmd/pub-lish.html
Page source: https://github.com/dart-lang/site-www/tree/master/src/tools/pub/cmd/pub-lish.md

We need to remove the server discussion from this page.


Original description of issue:

The task pub publish can´t add the option --server

@kwalrath kwalrath added fix.bug Needs fix of incorrect copy, code, or visual e0-minutes Can complete in < 60 minutes of normal, not dedicated, work p2-medium Necessary but not urgent concern. Resolve when possible. t.cli-tools Relates to the dart command line tools act.wait-for-customer Needs response from customer labels Sep 1, 2020
@kwalrath
Copy link
Contributor

kwalrath commented Sep 1, 2020

@jonasfj can you (or someone you know :) clarify the situation with the --server option, and whether it should be mentioned in https://dart.dev/tools/pub/cmd/pub-lish and the help?

@jonasfj
Copy link
Member

jonasfj commented Sep 1, 2020

This is a problem in flutter_tool wrapping pub.

You can do it with flutter pub pub publish --server ..., but this option isn't very useful, unless you're publishing to a server behind VPN as we don't have any authentication in place for this.

Another option is to use the publish_to: <server> in pubspec.yaml. Again, then authentication story for these is not good.


I would be okay with not documenting this option on dart.dev, given that we don't have a good story with authentication for 3rd-party pub servers, I think it's reasonable that we don't document it too much.

Also there is afaik 3 ways to do this:

  • pub publish --server <url>,
  • publish_to: <url> in pubspec.yaml, and
  • export PUB_HOSTED_URL=<url> && pub publish.

Neither of these supports authentication, so users that do this typically rely on network security by being behind a VPN.

The situation is not ideal, there are issue for this: dart-lang/pub#1381

@kwalrath kwalrath changed the title pub pubish can´t add server option Remove server option from pub publish page Sep 1, 2020
@kwalrath kwalrath added co.beginner Community opportunity to make a small change to documentation co.help-wanted Community opportunity to help documentation and removed act.wait-for-customer Needs response from customer labels Sep 1, 2020
Aman9026 added a commit to Aman9026/site-www that referenced this issue Sep 10, 2020
 removed server option from pub publish page to fix dart-lang#2612
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
co.beginner Community opportunity to make a small change to documentation co.help-wanted Community opportunity to help documentation e0-minutes Can complete in < 60 minutes of normal, not dedicated, work fix.bug Needs fix of incorrect copy, code, or visual p2-medium Necessary but not urgent concern. Resolve when possible. t.cli-tools Relates to the dart command line tools
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants