-
Notifications
You must be signed in to change notification settings - Fork 24
fix(node): gate /ipfs/pins and /arweave/anchors behind authentication (#121) #134
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
Open
Gravirei
wants to merge
70
commits into
Gitlawb:main
Choose a base branch
from
Gravirei:bug_fix_2
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
70 commits
Select commit
Hold shift + click to select a range
4506480
fix(node): gate GET /ipfs/{cid} on reachable allowed-set, not deny-se…
Gravirei 3aa7bf0
perf(ipfs): check object existence before allowed-blob walk
Gravirei 63580c7
refactor(ipfs): improve formatting and readability in get_by_cid func…
Gravirei 002f354
refactor(ipfs): streamline object retrieval by separating type and co…
Gravirei f2c91a8
docs(ipfs): update get_by_cid comment to reflect split object retrieval
Gravirei 0b7c494
Merge remote-tracking branch 'upstream/main'
Gravirei 03ba714
Run cargo fmt on store.rs
Gravirei 03c90f8
Merge remote-tracking branch 'upstream/main'
Gravirei 94bb216
Merge remote-tracking branch 'upstream/main'
Gravirei 79650df
Merge remote-tracking branch 'upstream/main'
Gravirei 26d8d48
Merge remote-tracking branch 'upstream/main'
Gravirei 8c51c63
Merge remote-tracking branch 'upstream/main'
Gravirei 48661a6
fix(node): gate /ipfs/pins and /arweave/anchors behind authentication…
Gravirei f4cb936
test(node): add integration tests for /ipfs/pins and /arweave/anchors…
Gravirei f51538a
resolve findings
Gravirei 8edb71d
fix(node): require authentication for global anchor and pin listings
Gravirei 5232f2b
test: format test_support.rs after CI failure
Gravirei 4e6294f
test: extract test helpers and add anchors anonymous coverage
Gravirei 496c839
fix(node): clamp negative anchor limit and add build_router pin integ…
Gravirei c6befaf
fix(node): filter global anchors before limit and use deduped repo vi…
Gravirei 15426c8
fix: resolve rebase conflicts and update pins path to use allowed_blo…
Gravirei e0fed8b
fix(node): resolve mirror rows in anchor per-repo path and bound glob…
Gravirei 542d877
fix(node): constrain anchor queries by owner_did to prevent cross-DID…
Gravirei f179512
fix(clippy): use is_some_and instead of map_or(false, ...)
Gravirei b58f713
fix(db,cli): filter arweave anchors in SQL and sign IPFS pin requests
Gravirei 027b7fe
test: add slug-collision regression test for global anchor query
Gravirei dfce033
fix: optimize global pin listing and verify CLI pin signatures
Gravirei 5e2b2ff
fix: address cargo clippy warnings
Gravirei 7882f9a
fix: clamp pin limits, normalize owner DIDs, and add migration v10 up…
Gravirei 4e12cde
Potential fix for pull request finding
Gravirei e458bdb
fix(node): apply Copilot review suggestions on PR 134
Gravirei 9c4f85e
fix(node): add anchors build_router test, v10 orphan test, v11 index …
Gravirei 7f07774
fix: include anchored_at DESC in arweave_anchors composite index
Gravirei 620321d
fix(node): over-fetch pins in batches and post-filter path-scoped hid…
Gravirei 5f6bf3c
fix(node): keyset-cursor pagination for pins, cap walks, add over-fet…
Gravirei 071fc81
fix(node): early return for zero pins limit, fail-open after walk cap
Gravirei 71e52fa
fix(node): fail closed after walk cap, bound keyset scan with next_cu…
Gravirei 45660e5
fix(node,gl): derive next_cursor from visible pins, opaque base64, CL…
Gravirei 69aea88
fix: address PR reviews for IPFS pagination and CLI options
Gravirei 766e114
fix: format code and correct test logic for IPFS cursor guard
Gravirei 7dcf606
fix: correct INSERT INTO visibility_rules column names in test
Gravirei e1c227e
fix(test): split multi-statement INSERT in test_ipfs_cursor_guard
Gravirei 27805db
fix(node,db): remove legacy-pin compat path, add truncated_cursor resume
Gravirei ccb4a82
fix(node,gl): opaque truncated_cursor via server-side cache, follow i…
Gravirei e7fd7ea
fix(node,gl): walk-budget truncation, retry-safe cursors, advancement…
Gravirei 87d1718
fix: remove unused has_keypair method from NodeClient
Gravirei 53b607f
fix: address P1/P2 review findings - remove legacy pin leak and in-me…
Gravirei fbbf46c
fix: address P1/P2 review findings - opaque truncated cursor and norm…
Gravirei 5244351
fix: P1+P2+P3 - AEAD opaque cursor, HKDF subkey, rate-limit, paginati…
Gravirei 5952e6a
fix: return 400 on malformed ordinary cursor; reify GL pagination loops
Gravirei 7bb5f00
fix: resolve node rate limiter and dashboard pins issues
Gravirei 7368582
fix: enhance advancement guard in IPFS pin fetching to handle consecu…
Gravirei 28fd4a4
fix: P2+P3 - pagination bounds, non-blob pins, deny tests, TTL, MAX_W…
Gravirei cd070bd
fix: cache repo path in allowed_blobs_by_repo to avoid double acquire…
Gravirei 30eea43
fix: remove all_blobs.is_empty guard so zero-blob repos (e.g. empty c…
Gravirei 21e8026
fix: batch structural-object probes, fix MAX_WALKS completeness gap, …
Gravirei ae8da4c
fix: resolve merge conflict in test_support.rs
Gravirei 5434c1a
fix: add missing closing brace for anchors_global_slug_collision_regr…
Gravirei f0d3701
fix: set git author/committer env vars in structural pin test for CI
Gravirei 2ed2507
fix: address P1/P2/P3 findings
Gravirei a2f25a1
fix: correct P3 findings — timeout comment and expired-token restore
Gravirei 370962e
fix: remove redundant borrow in format! arg (clippy)
Gravirei 0972bff
fix: refactor pagination into list_pins_paginated, add 400-restore test
Gravirei 289b61b
fix: remove redundant borrow in println (clippy)
Gravirei f25ee5f
fix: address review findings — expand/contract v10 migration, paginat…
Gravirei 41df3bf
Merge remote-tracking branch 'upstream/main' into bug_fix_2
Gravirei e488690
fix: correct v10 migration expand/contract — keep old PK, add UNIQUE …
Gravirei 83078f9
fix: address 5 findings — migration per-repo writes, walk budget skip…
Gravirei 73e5326
fix: fmt and probe_limit rework — treat unprobed candidates conservat…
Gravirei 741cc11
fix: restore Unavailable for transport/rejection errors in node statu…
Gravirei File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.