Transaction context should be merged into the evaluation context of the hooks

[kouk]

Hi,
During flag evaluation the transaction context is merged into the evaluation context here:

python-sdk/openfeature/client.py

Line 778 in 288bd6b

merged_context = self._before_hooks_and_merge_context(

However the hook_context.evaluation_context is created before this, so it doesn't contain the attributes or targeting key that exists in the propagated transaction context. This is unexpected. I believe that the _before_hooks_and_merge_context should be called inside _establish_hooks_and_provider so that the evaluation_context field in the hook_context can be up to date with all the attributes.

[beeme1mr]

Hey @kouk, I agree that context should be merged prior to calling the before hook. I checked the JavaScript implementation and it behaves as you described. Would you be willing to make this change?

[kouk]

@beeme1mr sure. I can take a stab at it.