Lukáš Procházka

Results 15 comments of Lukáš Procházka

In [RequestExecutor.cs](https://github.com/ChilliCream/hotchocolate/blob/main/src/HotChocolate/Core/src/Execution/RequestExecutor.cs) on line 78 the `_requestDelegate` containing middlewares is called. In [BatchExecutor.Enumerable.cs](https://github.com/ChilliCream/hotchocolate/blob/main/src/HotChocolate/Core/src/Execution/Batching/BatchExecutor.Enumerable.cs) the `_requestDelegate` is not called.

Okay, my mistake. Middlewares would be called but execution fails on #5223 execption before middlewares get called (by recursively calling `IRequestExecutor`). Middlewares should be called BEFORE line 74 in [BatchExecutor.Enumerable.cs](https://github.com/ChilliCream/hotchocolate/blob/main/src/HotChocolate/Core/src/Execution/Batching/BatchExecutor.Enumerable.cs)...

@michaelstaib I have inspected the issue reported by code analyzer, it seemst that it is caused by tests in namespace `StrawberryShake.CodeGeneration.CSharp.Analyzers.CustomScalar` as there is ``` public static class DependencyInjection {...

I signed CLA for both accounts (I am the other account aswell). Can someone else do the second approval? **Some feeback:** This agreement does not take conflict of laws (also...

Last words: Besides lex incorpationis (and lex patriae in case of people) not being used that much in contract law in favour of lex domicili (or lex loci solutionis), you...

@michaelstaib can you cherry-pick the this merge to main? (or I can create another PR)

When this change will propagate to the live website?

I came across this issue in 12.9.0. BatchExecutor does not work with persisted queries.