Axway API integration

[lpriima]

Auto instrumentation for axway api.
It's only tested manually:

There is no way I'm aware of, how I can extract HTTP library form apigateway to create unit test. Moreover, com.vordel.dwe.http.ServerTransaction and other parts of axway http plugin are java frontend to native closed source implementation.

Todo:

• http status code extraction
• integration test

[dougqh]

This form of forName will initialize the class. It is safer to use the other forName(name, initialized, ClassLoader) instead with initialized=false.

[lpriima]

Thx. Updated

[dougqh]

Can we not just return null? Or does, the caller not support that?

[lpriima]

yes: https://github.com/DataDog/dd-trace-java/blob/master/dd-java-agent/agent-bootstrap/src/main/java/datadog/trace/bootstrap/instrumentation/decorator/HttpServerDecorator.java#L64

[dougqh]

If this happens, should we stop trying to the URL extraction?

[lpriima]

I think this exception should never happen. At the point we instrumenting ServerTransaction it should have some valid URI behind it: otherwise it wouldn't be able to reach the server where we instrumenting it.

[dougqh]

Overall, I think this looks good. My only concern is what happens if we do hit one of the cases where the method handle fails.

At that point, I think we start spamming the log. Admittedly, I think the odds of that are low.

[lpriima]

Overall, I think this looks good. My only concern is what happens if we do hit one of the cases where the method handle fails.

At that point, I think we start spamming the log. Admittedly, I think the odds of that are low.

I think it shouldn't happen because I don't think we can get broken URI at server side. And this is debug level logs anyway.

I've re-tested latest changeset:

[devinsba]

It's unfortunate we can't write tests for this yet, but it's low risk given that it should only activate against Axway