Give up when there is no open segment and LOG_ERROR is configured

[pfreixes]

Without setting this flag as False, any HTTP call done with the Aiohttp
Client when there is no an active segment will fail with the
SegmentNotFoundException exception.

This issue can be annoying for this systems that do not create
explicitly the segements and relay on the sampled incomming requests,
which create automatically segments just for a sub set of the incomming
request.

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[pfreixes]

Please consider this as a potential bug that might require a minor release.

[pfreixes]

Closing, the segment is always created by the middleware ... If I've found something different I will reopen it. Sorry for the noise

[pfreixes]

Reopening, worth considering the option of giving the option to the user to not break the request if any segment is present.

[haotianw465]

Hi, did you check AWS_XRAY_CONTEXT_MISSING on this page? https://docs.aws.amazon.com/xray/latest/devguide/xray-sdk-python-configuration.html

If the aiohttp client errors out due to segment is None in case of LOG_ERROR, then the aiohttp client patcher code needs to be fixed so it uses the original function when segment is not present.

[pfreixes]

You are right. In any case, PR fixed to take into account that Aiohttp client tracing must give up silently when the context_missing is configured as LOG_ERROR.

So, yes it needs to be fixed yet.

[pfreixes]

Any chance to have a minor release?

[haotianw465]

Yes we will get a minor release as soon as possible. Thanks for your patience.

[pfreixes]

@haotianw465 please consider this PR #59 also as a bug, therefore a candidate for the minor release.