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.

Sign up for GitHub

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

Jump to bottom

Update SSH for Access for Infrastructure dev docs #17415

Merged
merged 11 commits into from
Oct 22, 2024
Merged

Update SSH for Access for Infrastructure dev docs #17415

merged 11 commits into from
Oct 22, 2024

Conversation

asamborski
Copy link
Contributor

@asamborski asamborski commented Oct 8, 2024 •
edited
Loading

Summary

Updates to the SSH for Access for Infrastructure include:

  • Specifying support for SCP and rsync. SFTP is not yet supported.
  • Add to the SSH guide that proxy mode with WARP must be enabled.
  • Add a note about AL1 and need to modify SSH CA certificate permissions.
  • Correct typo in Networks.
  • Correct hostname character length.
  • Modify ssh-proxy-ca instructions to include the different scenarios to use POST vs. GET as well as amending instructions to include what is necessary if using an API Token and not a Global API key.
  • Remove step to install Cloudflare root CA as a prerequisite. This is no longer needed.

@asamborski asamborski requested review from janani-cr, ranbel, a team and kennyj42 as code owners October 8, 2024 20:52
@github-actions github-actions bot added the size/s label Oct 8, 2024
@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Oct 8, 2024 •
edited
Loading

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1407f2c
Status: ✅  Deploy successful!
Preview URL: https://de0e66d0.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://asamborski-more-ssh-enhancem.cloudflare-docs-7ou.pages.dev

View logs

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

Files with changes (up to 15)

Original Link Updated Link
https://developers.cloudflare.com/cloudflare-one/connections/connect-networks/use-cases/ssh/ssh-infrastructure-access/ https://asamborski-more-ssh-enhancem.cloudflare-docs-7ou.pages.dev/cloudflare-one/connections/connect-networks/use-cases/ssh/ssh-infrastructure-access/
https://developers.cloudflare.com/cloudflare-one/applications/non-http/infrastructure-apps/ https://asamborski-more-ssh-enhancem.cloudflare-docs-7ou.pages.dev/cloudflare-one/applications/non-http/infrastructure-apps/

@ranbel ranbel enabled auto-merge (squash) October 22, 2024 16:42
@ranbel ranbel merged commit 418aa0b into production Oct 22, 2024
13 checks passed
@ranbel ranbel deleted the asamborski_more_ssh_enhancements branch October 22, 2024 16:52
harshil1712 pushed a commit that referenced this pull request Dec 3, 2024
@asamborski @ranbel @harshil1712
Update SSH for Access for Infrastructure dev docs (#17415)
9342042 
* initial commit

* More updates to SSH for Access for Infrastructure

* Update ssh-infrastructure-access.mdx

small wording change

* update API commands

* move supported SSH commands

* Responding to comments and removing unneeded step to install CF root CA

* Apply suggestions from code review

* style guide update

---------

Co-authored-by: Ranbel Sun <ranbel@cloudflare.com>
Co-authored-by: ranbel <101146722+ranbel@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ranbel ranbel ranbel approved these changes

@janani-cr janani-cr Awaiting requested review from janani-cr janani-cr is a code owner

@kennyj42 kennyj42 Awaiting requested review from kennyj42 kennyj42 is a code owner

Assignees

@kennyj42 kennyj42

@ranbel ranbel

@janani-cr janani-cr

Labels
product:cloudflare-one size/s
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@asamborski @ranbel @kennyj42 @janani-cr