search

eclipse / microprofile-context-propagation

Apache License 2.0
33 stars 23 forks source link

work around mpconfig dropping support for empty array #198

Closed njr-11 closed 3 years ago

njr-11 commented 3 years ago

contains most of the fix for #160

njr-11 commented 3 years ago

LGTM, but should we forbid ThreadContextProvider with the name None to be extra sure we'll never run into issues? I guess MP config will always filter them out, but still…

It seems extremely unlikely that anyone would name their ThreadContextProvider "None", but it's easy enough to explicitly forbid that so we might as well. I'll add it in a separate pull, #199 , under the same issue given that there are already 3 approvals on this one.