Thank you for taking time to contribute this pull request! You might have already read the [contributor guide][1], but as a reminder, please make sure to:

  • Sign the contributor license agreement
  • Rebase your changes on the latest main branch and squash your commits
  • Add/Update unit tests as needed
  • Run a build and make sure all tests pass prior to submission

Comment From: tzolov

@williamspindox thank you for the contribution It would help to provide some context before and with the PR?

Comment From: williamspindox

When can you approve my pull request? Thank you

Comment From: markpollack

Apologies, it is a hard area for me to come up to speed and we let this slip. I'm scheduling for the M2 release now.

Comment From: markpollack

I'm not sure what was intended for this PR. The resulting changes don't have anything to do with MS Entra authentication and the comment that referred to this code

        -Assert.hasText(connectionProperties.getApiKey(), "API key must not be empty");
        -Assert.hasText(connectionProperties.getEndpoint(), "Endpoint must not be empty");
        +HttpLogOptions options = new HttpLogOptions();

also isn't related to auth, but seems incomplate and not there in the final PR.

The resulting change the PR makes is related to where the onxx model is loaded from (http or classpath).

I don't know why using http is better over classpath. I'm closing this PR and wont' merge. Please open up a new PR for any of these specific issues with a description explaining the purpose.