From 5fc2644c680d04f8f56df3f33562764f7155e922 Mon Sep 17 00:00:00 2001 From: Leonard Lyubich Date: Fri, 16 Sep 2022 19:09:25 +0400 Subject: [PATCH] [#416] session: Support new format of object session context Signed-off-by: Leonard Lyubich --- netmap/grpc/types.pb.go | Bin 45526 -> 45734 bytes object/grpc/service_grpc.pb.go | Bin 31238 -> 31238 bytes session/convert.go | 19 +++++++---- session/grpc/types.go | 9 +++-- session/grpc/types.pb.go | Bin 53262 -> 56844 bytes session/marshal.go | 18 ++++++---- session/test/generate.go | 2 +- session/types.go | 58 ++++++++++++++++++++++++++++++--- 8 files changed, 83 insertions(+), 23 deletions(-) diff --git a/netmap/grpc/types.pb.go b/netmap/grpc/types.pb.go index 265bc243c8b6b6316641176ecd846316d33b1d04..c34f6bb0ebaf2244b9d4017399f9cc1349e6ee5e 100644 GIT binary patch delta 219 zcmX|*OA5k342D5)<0-Ne1lu!+3NCyg;@ZYG#lUo?Oj4>V58)kLxe&aI7ZC5^_-YnE z{Dgd8?vK{_*}6u_DcQ7-9`%PA^5 zD^RoUfPMf-4U<)g{*>Xj!)mfvPe)J*f$k_I1{H~r0Ou1Zod|CYHFvX0G?=U+TfmPK Z`$8pp)bNYpuy#2Vl~L|DtXnU+EMHj_O=kcA delta 14 WcmZ4Xl$<6cC*~M}CrPZMg)i0Bu@_m|oBwlW_p}PslMHqhCeAa6b%&92;*&OQM2$R6{ i!>6DmWanY^!{pW^v&}an!jKih{Q(Sm!OdcEAEW_uO_z`W delta 390 zcmZqs!r1nOaf3PgIyZmcVJj{V8 cE@#%`;+$NYWViWdL>No~k`Fd>#Ql>70N%lzT>t<8 diff --git a/session/convert.go b/session/convert.go index d7662b4..b9a93e6 100644 --- a/session/convert.go +++ b/session/convert.go @@ -674,7 +674,7 @@ func (c *ObjectSessionContext) ToGRPCMessage() grpc.Message { m = new(session.ObjectSessionContext) m.SetVerb(ObjectSessionVerbToGRPCField(c.verb)) - m.SetAddress(c.addr.ToGRPCMessage().(*refsGRPC.Address)) + m.SetTarget(c.cnr.ToGRPCMessage().(*refsGRPC.ContainerID), refs.ObjectIDListToGRPCMessage(c.objs)) } return m @@ -688,20 +688,25 @@ func (c *ObjectSessionContext) FromGRPCMessage(m grpc.Message) error { var err error - addr := v.GetAddress() - if addr == nil { - c.addr = nil + cnr := v.GetTarget().GetContainer() + if cnr == nil { + c.cnr = nil } else { - if c.addr == nil { - c.addr = new(refs.Address) + if c.cnr == nil { + c.cnr = new(refs.ContainerID) } - err = c.addr.FromGRPCMessage(addr) + err = c.cnr.FromGRPCMessage(cnr) if err != nil { return err } } + c.objs, err = refs.ObjectIDListFromGRPCMessage(v.GetTarget().GetObjects()) + if err != nil { + return err + } + c.verb = ObjectSessionVerbFromGRPCField(v.GetVerb()) return nil diff --git a/session/grpc/types.go b/session/grpc/types.go index 64133a8..7cdb7d5 100644 --- a/session/grpc/types.go +++ b/session/grpc/types.go @@ -65,9 +65,12 @@ func (m *SessionToken_Body) SetContainerSessionContext(v *ContainerSessionContex } } -// SetAddress sets address of the object related to the session. -func (m *ObjectSessionContext) SetAddress(v *refs.Address) { - m.Address = v +// SetTarget specifies objects involved in the object session. +func (m *ObjectSessionContext) SetTarget(cnr *refs.ContainerID, objs []*refs.ObjectID) { + m.Target = &ObjectSessionContext_Target{ + Container: cnr, + Objects: objs, + } } // SetVerb sets type of request for which the token is issued. diff --git a/session/grpc/types.pb.go b/session/grpc/types.pb.go index 81b3f048d172dc8244fe57a6cf3d0300e86fbdcc..c964738d16746dd132051d439f6e295f092de22f 100644 GIT binary patch delta 4291 zcmai1dr+L!6(0yJ>}HejTwYo7@hAjl1N#6LVk2gRU|tRB+EFGZu)tzA!tS!WyxBk= z5(0q*erOPaK%QVm^B@{q8-k8)9it{4+d8eSgGTL4r<0PI>e#BW=icvIz}RX3f%~0% z?>Xo9JHK=G`+CQh9sA|7ixc5(JIeQ2D%^t0;&NH-HoR42o-jx zllH5dgetqU#!@MmZIyJGPHe6c%Iy5It4tx!%oM7umg-8u>Jn^rw_vGpxSKSBd#{Cj z?zJrVH#;2FR*Orp+sLKcE|_T3Wf3gZmKuxA?b2{hb}~8}tS$>VunXquYC#GlR5|T6 zqCbbF!b%Ymj*>~sT8%nR#H4XJ?QXjvQP}6Q+tw$FMyz-= z1}7p_SiU-Bx+o$nz;`b{IL5d9I7;nX{&tgU`i1DcjfRKKPA7*U5+GP@_4ev|%El_f z74!9>&QQ3U9GFl<1@f%0JFLBJv#ruvVRjQ04STH>dlzARk1k@<<#1Zel|)2_u!%G5 zD-BhFv&AYRm5C^L*hhzu{-o1Uk+JZ+xj-o66qMIhtxrT>oD6Esa`^&%n2C#S)7WdQ zZeF`Yv8eZ_=_|z2ntngY9TFvGCzZ94%H+0G@;s@OBq*2L?p);QCl>oA(<1gt!D*?o zI4!mcI#S*wlr3aeCd!KdlF|yzM3^KQQK@^)^$S|%N;SLbs4taWyG1Ui3k#R-7qH?V&9VIHQVCZzVFI~ z$j9+i>1X)6YVpd>%rvA$Ej}N}d;-PGt#O}GzopIHdxb-}GoQPa0easu#}wI1F*f5!X!zMM9feHrYp)8cND1CE+V{Kfj^ zGM#oeCX&>6XWOq)To#I3yH-wr@KjP@aBj9n&^Brf)Bd$<{PkHm)U)(x=EfZM*OnevG$A_ z<;?{+dL|KXHRogaND<~o0x{CQLMBpxGH0^C(EvwN6jZ&@e!2Mk@FVzkObz=1SzwNy z(J_8F7QF}J@x{~etBeMQUC-=9sTt;$7`!u~K&xA`k~Xq<2H92Y+DwZY@4IuDS?dEt z<~6NqWYvjgewwl-b7P%}JK#0R(lOH?>r7~BRWjPw*YjP8l03FMvecwAGE-@79JCbm zurpPxl>vPX0`55Fc(6;rhqX$4)}X=IF#SLN5Oz04;`$NpzrE3fR&n`pSj1_ZWDWyTtvWi{Lsp7)pDqGo* zf_u$Ml(Zx;3IjL|ZSFK&YDwo*aH_L7*f}zXwQ{qT>sTc1+qz6*D4CrWDb6;q_~%Wsk87D}0cKAmvfK6es9OoI zHxP~O#hgZw2ac_}DV4Y1#jybVySs{n7l*H#dBC9qCUhR+%YV^w(xXK1b5%I>befbX z-&%dhgph;sK&lOa_t4W<$N2^WUF?1J|+Mj0x`!wDrnZ;VJgO zkv#OA&p>HII=KuFVrWSM3oho4#36b>iQo5!;E_YAgh7EvIwQg;v{X7KGn!vo)OwO| zuQLp%I%A>k;^1y{nsBL8jR%evV8c;v>N#q{NH-C|<@s@M3jQ%7Lu^+n8VTXrGojei zRlsFf!N3@^8DF7iqHw1(gtnBp-Ni*t?M>Ucm3+Hf?8{sZpWnDUxEVip@^3 zc)s2WCHco=?iqod36^@sh_TuzOdZR|E6;}FpZntpkrKZ<76D^V7|TkhWihyh@K}#1 zVVrgw8!hT&a?g?fA`cLFj(+LC5Dljmy4%@n;@#$a~?F@89 zpauE7pSg=19)IItEIvUVhE9b-GsqiAEKweBpsqm^_E0@cSj!7O&}|^zxe6Vf2}^p5 zl}9FZ?;2luq_*)MD`9|h__UZmR)VXYDwOtY<;Du81l@5Hk|#s`4DfqCN@8Ay{>f~d z>rIiEXL8kYnwBu%F_bVc$;qA@kHf8@TwEm8&w5G}TxX;??F5+&NK*vePnbAhx%RZsGST7uA3BbP4r~ z3=JJB%$;3Fl&;3D6K4GCbPWFV{DZu9d~lGQ3D*T;X1n8Qkn76oBqQS3t#sc0iwnp>7bKDC7aDiAOu9nhM zqWzQt?V}L^Ofk4Q7>$dgh43_m(Top8d|L|nOGn`862%hYZalx8^{@CFPA_-kLunE- zApv|~6(5b0#LBn{qu*#YM+oy-7Le;3(8Y3RR_gueHJrgy*8x(pj-kt_K7yE|>I z&03wN!Pl4Kp?*0sNVF=D@NNYDb}80Rhn>HOz}!qI(l2M?@XsS@P5QYUH2u+KmtPQn zgWg<#jTgdk@p90>!?0^6(_gQF{kjIP&g}FvE^KJ87m#C- zjxdZ|e^@Er-KEc4ug^omtQtYHEAKa^-)~T%W>)pzj0#+sjr3!VRMQ{M$`y!yWAQ6! p_nX&n`Nl4M@%AiM)5-_|H5Nt3&9wY%f3!-71rFE31oleIVjTr2fRh~rwg-}2Q z1tdR|pMN`y~7cyFFSs2uW}qz?LP3fqo_({KX|ahQCTSz3CO$ywr zkASC6#-|7L7N}i2(KQ<<%lIGu@g~Fn4GPIGBLqh3VjzzAw!yYeD1f!qgmPyJU5=9F zNUI#*I^|(3Nfu8}orR4yX38+p^E3mL{D(=eBc>YxQvh8SF{Qq16FmV;P~55Ah|&{k z+;j=B{H_$y$HmC*mLm*u_-lfZ(4=C3%?4LbBE+>K9BtB~uYXb2y}|s3s%k z8Wkuwl|$W~)ydEdCgJN-GN{@k5z>~Aw4*PhvzAYY*n*l)BOcE35!TM_!;d zw`1$H1Pv21_`Q61+RaEAOGMXHHcpR)Vr5o}S58|a8NsZw0p^*OGO#);!zZWB7&;+= z@XQ{To~fAWEDJw0YvAnFk~}mdU@UkY47j^Nh3pQN_3chAraFu;xhpU=V8v>ih-9z8 zLg$Mtl$nXaWI$QRM*OK$jh$UA1V!84mBn4hY0{2zcM?`zLOkqBri?UnZ^o{(nMiDS z0sbat(j2$pZmo#!dTLboS$7WOBEydS6T9%w?l25YN5bECj3_~kJKa+1srn!iU2Ssg zX;woCDeUz-Q3)m8@^|Lr7C0$$samKKet-=h^+<5BM~aG`Y$g<{anfT{!;3wf(z&j* z5GE0S&KAlj<|>P_ehuP!`ItVp8AZLZSUqRu_$Z=LEG7(llSr+C$c6~X=auL>&u7=r zzSAAcJV;``cix8jb7|PrYvknapmKNiW)i8#l6u5JUME3kA0PhV@DO@C6*D};ud@d; zsWx~QUuXByyy-}RrY{yjeVOboGfRdwZ+;)wg9gseh~YW~uJ*ASmk*HJ+G3E^pNz$s z(Djs|!~HgV?n*%O;1)bLAj3_c82W(%q|a-J4l>aPG5$JG5Jj`%m@^VckB3?j8ZKzi zGN__O_2lqOPq4(tDzFS1V@#)r&ZYwU1 zvL>7xQ{vWW9?Z2F_|0fIZq5o(GA5&5bZcNNkD=5PmI5gMGG=3Cv1FieL4i-_)r7B2 z92YEO3uV+hZXXfV~k=XlG;gBdkl?wvOw zc07eikfR_4|2iea`{S>%MtFofe$GzjYY5l)0eTQVy(K_`60c51V#h>28NRUr_)=GW z0>qO!YxpUd%wwEC#%jc2UnptfxByQ2>Z9|(N;nYXo2dde`1un=%RVcK9fRAxJXie& zoO(}(A4ft+yKMR|;F2z@Ss??Oq+R~XWlq6?(P*YRFcd?5NrC0dSxl%FW;8M_k}W#I z?=}hX(*+flyVB6RVC2-nBu?XCWr2+j->wg;*YR~4VYR&i<$`!De=YPq_;t;Cd{QF8 zmsgbd-EZaCwKBjmu*08(cz*=y{9c{`+H2{--1lB3?AM}1YyB~J?HiuX=lu9`a4^A6VjHmSLz;mU z=IiMptNGLZYBj2UsgfM0kLV@9#P|l@zp2g=`iWfrrnFz}z8}MM|3VK)el8?%8D8E?b SW%_P?v4=+xrF``V-oF70%=df% diff --git a/session/marshal.go b/session/marshal.go index d7db4d2..2c82179 100644 --- a/session/marshal.go +++ b/session/marshal.go @@ -21,8 +21,8 @@ const ( lifetimeNotValidBeforeField = 2 lifetimeIssuedAtField = 3 - objectCtxVerbField = 1 - objectCtxAddressField = 2 + objectCtxVerbField = 1 + objectCtxTargetField = 2 sessionTokenBodyIDField = 1 sessionTokenBodyOwnerField = 2 @@ -210,10 +210,11 @@ func (c *ObjectSessionContext) StableMarshal(buf []byte) []byte { buf = make([]byte, c.StableSize()) } - var offset int - - offset += proto.EnumMarshal(objectCtxVerbField, buf[offset:], int32(c.verb)) - proto.NestedStructureMarshal(objectCtxAddressField, buf[offset:], c.addr) + offset := proto.EnumMarshal(objectCtxVerbField, buf, int32(c.verb)) + proto.NestedStructureMarshal(objectCtxTargetField, buf[offset:], &objectSessionContextTarget{ + cnr: c.cnr, + objs: c.objs, + }) return buf } @@ -224,7 +225,10 @@ func (c *ObjectSessionContext) StableSize() (size int) { } size += proto.EnumSize(objectCtxVerbField, int32(c.verb)) - size += proto.NestedStructureSize(objectCtxAddressField, c.addr) + size += proto.NestedStructureSize(objectCtxTargetField, &objectSessionContextTarget{ + cnr: c.cnr, + objs: c.objs, + }) return size } diff --git a/session/test/generate.go b/session/test/generate.go index 25976bc..8bb2c82 100644 --- a/session/test/generate.go +++ b/session/test/generate.go @@ -195,7 +195,7 @@ func GenerateObjectSessionContext(empty bool) *session.ObjectSessionContext { if !empty { m.SetVerb(session.ObjectVerbHead) - m.SetAddress(refstest.GenerateAddress(false)) + m.SetTarget(refstest.GenerateContainerID(false), refstest.GenerateObjectIDs(false)...) } return m diff --git a/session/types.go b/session/types.go index dfa8c5e..cb14825 100644 --- a/session/types.go +++ b/session/types.go @@ -4,6 +4,7 @@ import ( "github.com/nspcc-dev/neofs-api-go/v2/acl" "github.com/nspcc-dev/neofs-api-go/v2/refs" "github.com/nspcc-dev/neofs-api-go/v2/status" + "github.com/nspcc-dev/neofs-api-go/v2/util/proto" ) type CreateRequestBody struct { @@ -40,10 +41,48 @@ type TokenLifetime struct { type ObjectSessionVerb uint32 +type objectSessionContextTarget struct { + cnr *refs.ContainerID + + objs []refs.ObjectID +} + +const ( + _ = iota + fNumObjectTargetContainer + fNumObjectTargetObjects +) + +func (x objectSessionContextTarget) StableMarshal(buf []byte) []byte { + if buf == nil { + buf = make([]byte, x.StableSize()) + } + + offset := proto.NestedStructureMarshal(fNumObjectTargetContainer, buf, x.cnr) + + for i := range x.objs { + offset += proto.NestedStructureMarshal(fNumObjectTargetObjects, buf[offset:], &x.objs[i]) + } + + return buf +} + +func (x objectSessionContextTarget) StableSize() (size int) { + size += proto.NestedStructureSize(fNumObjectTargetContainer, x.cnr) + + for i := range x.objs { + size += proto.NestedStructureSize(fNumObjectTargetObjects, &x.objs[i]) + } + + return size +} + type ObjectSessionContext struct { verb ObjectSessionVerb - addr *refs.Address + cnr *refs.ContainerID + + objs []refs.ObjectID } type TokenContext interface { @@ -617,16 +656,25 @@ func (c *ObjectSessionContext) SetVerb(v ObjectSessionVerb) { c.verb = v } -func (c *ObjectSessionContext) GetAddress() *refs.Address { +func (c *ObjectSessionContext) GetContainer() *refs.ContainerID { if c != nil { - return c.addr + return c.cnr } return nil } -func (c *ObjectSessionContext) SetAddress(v *refs.Address) { - c.addr = v +func (c *ObjectSessionContext) GetObjects() []refs.ObjectID { + if c != nil { + return c.objs + } + + return nil +} + +func (c *ObjectSessionContext) SetTarget(cnr *refs.ContainerID, objs ...refs.ObjectID) { + c.cnr = cnr + c.objs = objs } func (t *TokenBody) GetID() []byte {