Skip to content

Commit 6c31225

Browse files
whyrusleepingwhyrusleeping
committed
refactor resolved to allow transparent resolution of unixfs sharded paths
License: MIT Signed-off-by: Jeromy <[email protected]>
1 parent 259aeb4 commit 6c31225

File tree

22 files changed

+162
-48
lines changed

22 files changed

+162
-48
lines changed

‎core/builder.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ func setupNode(ctx context.Context, n *IpfsNode, cfg *BuildCfg) error {
179179
// this is kinda sketchy and could cause data loss
180180
n.Pinning = pin.NewPinner(n.Repo.Datastore(), n.DAG)
181181
}
182-
n.Resolver = &path.Resolver{DAG: n.DAG}
182+
n.Resolver = path.NewBasicResolver(n.DAG)
183183

184184
err = n.loadFilesRoot()
185185
if err != nil {

‎core/commands/files/files.go

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import (
1515
mfs "github.com/ipfs/go-ipfs/mfs"
1616
path "github.com/ipfs/go-ipfs/path"
1717
ft "github.com/ipfs/go-ipfs/unixfs"
18+
uio "github.com/ipfs/go-ipfs/unixfs/io"
1819

1920
logging "gx/ipfs/QmNQynaz7qfriSUJkiEZUrm2Wen1u3Kj9goZzWtrPyu7XR/go-log"
2021
context "gx/ipfs/QmZy2y8t9zQH2a1b8q2ZSLKp17ATuJoCNxxyMFG5qFExpt/go-net/context"
@@ -259,7 +260,12 @@ func getNodeFromPath(ctx context.Context, node *core.IpfsNode, p string) (*dag.N
259260
return nil, err
260261
}
261262

262-
return core.Resolve(ctx, node, np)
263+
resolver := &path.Resolver{
264+
DAG: node.DAG,
265+
ResolveOnce: uio.ResolveUnixfsOnce,
266+
}
267+
268+
return core.Resolve(ctx, node.Namesys, resolver, np)
263269
default:
264270
fsn, err := mfs.Lookup(node.FilesRoot, p)
265271
if err != nil {

‎core/commands/get.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ may also specify the level of compression by specifying '-l=<1-9>'.
6363
}
6464
p := path.Path(req.Arguments()[0])
6565
ctx := req.Context()
66-
dn, err := core.Resolve(ctx, node, p)
66+
dn, err := core.Resolve(ctx, node.Namesys, node.Resolver, p)
6767
if err != nil {
6868
res.SetError(err, cmds.ErrNormal)
6969
return

‎core/commands/ls.go

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import (
1212
merkledag "github.com/ipfs/go-ipfs/merkledag"
1313
path "github.com/ipfs/go-ipfs/path"
1414
unixfs "github.com/ipfs/go-ipfs/unixfs"
15+
uio "github.com/ipfs/go-ipfs/unixfs/io"
1516
unixfspb "github.com/ipfs/go-ipfs/unixfs/pb"
1617
)
1718

@@ -71,7 +72,18 @@ format:
7172

7273
var dagnodes []*merkledag.Node
7374
for _, fpath := range paths {
74-
dagnode, err := core.Resolve(req.Context(), node, path.Path(fpath))
75+
p, err := path.ParsePath(fpath)
76+
if err != nil {
77+
res.SetError(err, cmds.ErrNormal)
78+
return
79+
}
80+
81+
r := &path.Resolver{
82+
DAG: node.DAG,
83+
ResolveOnce: uio.ResolveUnixfsOnce,
84+
}
85+
86+
dagnode, err := core.Resolve(req.Context(), node.Namesys, r, p)
7587
if err != nil {
7688
res.SetError(err, cmds.ErrNormal)
7789
return

‎core/commands/object/diff.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,13 +73,13 @@ Example:
7373

7474
ctx := req.Context()
7575

76-
obj_a, err := core.Resolve(ctx, node, pa)
76+
obj_a, err := core.Resolve(ctx, node.Namesys, node.Resolver, pa)
7777
if err != nil {
7878
res.SetError(err, cmds.ErrNormal)
7979
return
8080
}
8181

82-
obj_b, err := core.Resolve(ctx, node, pb)
82+
obj_b, err := core.Resolve(ctx, node.Namesys, node.Resolver, pb)
8383
if err != nil {
8484
res.SetError(err, cmds.ErrNormal)
8585
return

‎core/commands/object/object.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ is the raw data of the object.
8888
}
8989

9090
fpath := path.Path(req.Arguments()[0])
91-
node, err := core.Resolve(req.Context(), n, fpath)
91+
node, err := core.Resolve(req.Context(), n.Namesys, n.Resolver, fpath)
9292
if err != nil {
9393
res.SetError(err, cmds.ErrNormal)
9494
return
@@ -127,7 +127,7 @@ multihash.
127127
}
128128

129129
fpath := path.Path(req.Arguments()[0])
130-
node, err := core.Resolve(req.Context(), n, fpath)
130+
node, err := core.Resolve(req.Context(), n.Namesys, n.Resolver, fpath)
131131
if err != nil {
132132
res.SetError(err, cmds.ErrNormal)
133133
return
@@ -190,7 +190,7 @@ This command outputs data in the following encodings:
190190

191191
fpath := path.Path(req.Arguments()[0])
192192

193-
object, err := core.Resolve(req.Context(), n, fpath)
193+
object, err := core.Resolve(req.Context(), n.Namesys, n.Resolver, fpath)
194194
if err != nil {
195195
res.SetError(err, cmds.ErrNormal)
196196
return
@@ -257,7 +257,7 @@ var ObjectStatCmd = &cmds.Command{
257257

258258
fpath := path.Path(req.Arguments()[0])
259259

260-
object, err := core.Resolve(req.Context(), n, fpath)
260+
object, err := core.Resolve(req.Context(), n.Namesys, n.Resolver, fpath)
261261
if err != nil {
262262
res.SetError(err, cmds.ErrNormal)
263263
return

‎core/commands/object/patch.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ the limit will not be respected by the network.
7373
return
7474
}
7575

76-
rootnd, err := core.Resolve(req.Context(), nd, root)
76+
rootnd, err := core.Resolve(req.Context(), nd.Namesys, nd.Resolver, root)
7777
if err != nil {
7878
res.SetError(err, cmds.ErrNormal)
7979
return
@@ -135,7 +135,7 @@ Example:
135135
return
136136
}
137137

138-
root, err := core.Resolve(req.Context(), nd, rp)
138+
root, err := core.Resolve(req.Context(), nd.Namesys, nd.Resolver, rp)
139139
if err != nil {
140140
res.SetError(err, cmds.ErrNormal)
141141
return
@@ -193,7 +193,7 @@ Removes a link by the given name from root.
193193
return
194194
}
195195

196-
root, err := core.Resolve(req.Context(), nd, rootp)
196+
root, err := core.Resolve(req.Context(), nd.Namesys, nd.Resolver, rootp)
197197
if err != nil {
198198
res.SetError(err, cmds.ErrNormal)
199199
return
@@ -266,7 +266,7 @@ to a file containing 'bar', and returns the hash of the new object.
266266
return
267267
}
268268

269-
root, err := core.Resolve(req.Context(), nd, rootp)
269+
root, err := core.Resolve(req.Context(), nd.Namesys, nd.Resolver, rootp)
270270
if err != nil {
271271
res.SetError(err, cmds.ErrNormal)
272272
return
@@ -292,7 +292,7 @@ to a file containing 'bar', and returns the hash of the new object.
292292

293293
e := dagutils.NewDagEditor(root, nd.DAG)
294294

295-
childnd, err := core.Resolve(req.Context(), nd, childp)
295+
childnd, err := core.Resolve(req.Context(), nd.Namesys, nd.Resolver, childp)
296296
if err != nil {
297297
res.SetError(err, cmds.ErrNormal)
298298
return

‎core/commands/pin.go

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -270,8 +270,13 @@ func pinLsKeys(args []string, typeStr string, ctx context.Context, n *core.IpfsN
270270

271271
keys := make(map[string]RefKeyObject)
272272

273-
for _, p := range args {
274-
dagNode, err := core.Resolve(ctx, n, path.Path(p))
273+
for _, sp := range args {
274+
p, err := path.ParsePath(sp)
275+
if err != nil {
276+
return nil, err
277+
}
278+
279+
dagNode, err := core.Resolve(ctx, n.Namesys, n.Resolver, p)
275280
if err != nil {
276281
return nil, err
277282
}

‎core/commands/publish.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ func publish(ctx context.Context, n *core.IpfsNode, k crypto.PrivKey, ref path.P
135135

136136
if opts.verifyExists {
137137
// verify the path exists
138-
_, err := core.Resolve(ctx, n, ref)
138+
_, err := core.Resolve(ctx, n.Namesys, n.Resolver, ref)
139139
if err != nil {
140140
return nil, err
141141
}

‎core/commands/refs.go

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -196,8 +196,13 @@ var refsMarshallerMap = cmds.MarshalerMap{
196196

197197
func objectsForPaths(ctx context.Context, n *core.IpfsNode, paths []string) ([]*dag.Node, error) {
198198
objects := make([]*dag.Node, len(paths))
199-
for i, p := range paths {
200-
o, err := core.Resolve(ctx, n, path.Path(p))
199+
for i, sp := range paths {
200+
p, err := path.ParsePath(sp)
201+
if err != nil {
202+
return nil, err
203+
}
204+
205+
o, err := core.Resolve(ctx, n.Namesys, n.Resolver, p)
201206
if err != nil {
202207
return nil, err
203208
}

0 commit comments

Comments
 (0)