Skip to content
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

Better curl instruction #126

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Better curl instruction #126

wants to merge 1 commit into from

Conversation

smorimoto
Copy link
Contributor

No description provided.

@maiste maiste added the web Related to website design and development label Nov 14, 2024
Signed-off-by: Sora Morimoto <[email protected]>
@smorimoto
Copy link
Contributor Author

Gentle ping

@Leonidas-from-XIV
Copy link
Contributor

I still don't see how --proto="=https" is improving the security in our particular situation.

@smorimoto
Copy link
Contributor Author

It's better than nothing and it's the de facto standard.

@Leonidas-from-XIV
Copy link
Contributor

But my point is what problem is this solving that we are actually encountering? If the problem is just "cargo does it differently" then I don't see the point making our instructions more complex just to be as complex as cargo's.

I think it is a good idea to copy good solutions if they solve problems we also have (e.g. the shell function wrapper or the conditional path extension that was inspired by cargo et all), but we don't have the issue that --proto '=https' is preventing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
web Related to website design and development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants