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

chore: add references to ipfs-gateway.md #1922

Merged
merged 4 commits into from
Sep 10, 2024
Merged

Conversation

lidel
Copy link
Member

@lidel lidel commented Sep 3, 2024

  • Concept page should link to relevant sections of "Address IPFS on the web" tutorial.
  • Remove link to article which anti-pattern
  • CI green

@@ -66,19 +64,12 @@ _Private gateways_ are configured to limit access to requests from specific doma

They are frequently, but not exclusively, used behind firewalls. Running [IPFS Desktop](https://github.com/ipfs-shipyard/ipfs-desktop#ipfs-desktop) or another form of IPFS node triggers connection attempts to other IPFS peers. Private network administrators may treat such connection attempts as potential security vulnerabilities. Private IPFS gateway servers located inside the private network and running a trusted code base provide an alternative architecture for read/write access to externally-hosted IPFS content.

This [tutorial configuring an IPFS gateway on a Google Cloud platform](https://blog.stacktical.com/ipfs/gateway/dapp/2019/09/21/ipfs-server-google-cloud-platform.html) includes information on constraining access for a private gateway setup.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💭 Removed due to CI getting 403 + content promoting antipattern, exposes subset of RPC port to internet, anyone can upload.

@lidel lidel requested a review from 2color September 3, 2024 21:54
@lidel lidel marked this pull request as ready for review September 3, 2024 21:54
docs/concepts/ipfs-gateway.md Outdated Show resolved Hide resolved
docs/concepts/ipfs-gateway.md Outdated Show resolved Hide resolved
docs/concepts/ipfs-gateway.md Outdated Show resolved Hide resolved
@lidel lidel merged commit 5bc60f7 into main Sep 10, 2024
5 checks passed
@lidel lidel deleted the gateway-cross-reference-docs branch September 10, 2024 21:39
Copy link
Member

@2color 2color left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants