Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1184
I'm still not convinced this is a particularly useful API, but it's also
fairly benign. However, I do feel that if we are going to do this, we should
make the prototype something like:
```C
TSStatusCode TSClientRequestUuidGet(TSHttpTxn txnp, char* destination)
```
or something with a fixed size array, like
```C
TSStatusCode TSClientRequestUuidGet(TSHttpTxn txnp, char
destination[MAX_SIZE_OF_THIS_BAD_BOY)
```
Where the caller is in charge of allocating enough storage for
"destination". This allows it to in most cases be statically allocated (on the
stack).
Also, the patch as it is is not right, you can't return a pointer into the
arena, after you released it. But that's a moot point if we change the
signature as the above.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
