4 ms·
Would you rather a team move faster and be more productive or be a purist and disallow abstractions to avoid some potential runtime tracing challenges which can
by CharlieDigital 1y ago
Would you rather a team move faster and be more productive or be a purist and disallow abstractions to avoid some potential runtime tracing challenges which can be mitigated with good use of OTEL and logging? I don't know about you, but I'm going to bias towards productivity and use integration tests + observability to safeguard code.
- hu3 1y agoHow much faster are we talking? Because you'd have to account for the time lost debugging annotations.
- CharlieDigital 1y agoWhat are you working on that you're debugging annotations everyday? I'd say you've made a big mistake if you're doing that/you didn't read the docs and don't understand how to use the attribute. (Of course you are also free to write C# without any of the built in frameworks and write purely explicit handling and routing) On the other hand, we write CRUD every day so anything that saves repetition with CRUD is a gain.
- OtomotO 1y agoI don't debug them every day, but when I do, it takes days for a nasty bug to be worked out. Yes, they make CRUD stuff very easy and convenient.
- badhombres 1y agoDisallow bespoke abstractions and use the industry standard ones instead. People who make abstractions inflate how productive they’re making everyone else. Your user base is much smaller than popular libs, so your docs and abstractions are not as battle tested and easy to use as much as you think.
- CharlieDigital 1y agoThis is raw OpenFGA code: await client.Write( new ClientWriteRequest( [ // Alice is an admin of form 123 new() { Object = "form:124", Relation = "editor", User = "user:avery", }, ] ) ); var checkResponse = await client.Check( new ClientCheckRequest { Object = "form:124", Relation = "editor", User = "user:avery", } ); var checkResponse2 = await client.Check( new ClientCheckRequest { Object = "form:125", Relation = "editor", User = "user:avery", } ); This is an abstraction we wrote on top of it: await Permissions .WithClient(client) .ToMutate() .Add<User, Form>("alice", "editor", "226") .Add<User, Team>("alice", "member", "motion") .SaveChangesAsync(); var allAllowed = await Permissions .WithClient(client) .ToValidate() .Can<User, Form>("alice", "edit", "226") .Has<User, Team>("alice", "member", "motion") .ValidateAllAsync(); You would make the case that the former is better than the latter?
- badhombres 1y agoIn the first example, I have to learn and understand OpenFGA, in the second example I have to learn and understand OpenFGA and your abstractions.
- mrsmrtss 1y agoWell the point of using abstractions is that you don't need to know the things that it is abstracting. I think the abstraction here is self explaining what it does and you can certainly understand and use it without needing to understand all the specifics behind it.
- CharlieDigital 1y agoMore importantly: it prevents "usr:alice_123" instead of "user:alice_123" by using the type constraint to generate the prefix for the identifier.