Adding in additional header to determine a more stable isolation-group

uber-go / cadence-client

Framework for authoring workflows and activities running on top of the Cadence orchestration engine.

https://cadenceworkflow.io

MIT License

345 stars 131 forks source link

Adding in additional header to determine a more stable isolation-group #1252

Closed davidporter-id-au closed 1 year ago

davidporter-id-au commented 1 year ago

What changed?

Adds a new header for downstream which is expected to be more stable to determine the origin of the worker
Repurposes the auth / identity wrapper a bit to shim this in there
Why?

How did you test it?

Potential risks

davidporter-id-au commented 1 year ago

broadly: might be some stuff to clean up, but structurally sound I think. we already have wrappers, one more for another optional feature seems fine.

I'm not all that sold on these wrappers to begin with, but they're already here ¯_(ツ)_/¯

I'm not going to investigate codegen in this PR, though it certainly feel's like it should have it at this point. Cleaned up the test though