Skip to content

Add new API to allow authentication with Azure Identity#76

Merged
bainian12345 merged 3 commits into
devfrom
bailiu/azure-identity-tokenprovider
Oct 27, 2021
Merged

Add new API to allow authentication with Azure Identity#76
bainian12345 merged 3 commits into
devfrom
bailiu/azure-identity-tokenprovider

Conversation

@bainian12345
Copy link
Copy Markdown
Contributor

Allow the user to create TokenProvider object with a TokenCredential from AzureIdentity for authentication. This would support authentication with client secret, Azure managed identity, Azure user assigned identity, and many other methods shown here.

@bainian12345 bainian12345 requested a review from dlstucki October 27, 2021 00:29
Comment thread pom.xml
@@ -6,7 +6,6 @@
<groupId>com.microsoft.azure</groupId>
<artifactId>azure-relay</artifactId>
<version>0.0.3</version>
Copy link
Copy Markdown
Contributor

@dlstucki dlstucki Oct 27, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3

Will this get incremented as part of some upcoming release with these changes? #Closed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes definitely, I will do it right before actually releasing it.

@@ -0,0 +1,31 @@
package com.microsoft.azure.relay;
Copy link
Copy Markdown
Contributor

@dlstucki dlstucki Oct 27, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

package

Add a copyright header? #Closed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Damn, it seems like half the files don't have the header currently. I will add all of them.

Copy link
Copy Markdown
Contributor

@dlstucki dlstucki Oct 27, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, boy. I would have rather had touching all those unrelated files be part of a different PR that does only copyright headers. Now this PR is touching many dozens of files and looks far scarier than it should.

However, you've done the work now. No point in making you do more work to get to the same ending place.


// Initializes a new instance of the <see cref="TokenProvider" /> class.
/**
* Initializes a new instance of the TokenProviders class.
Copy link
Copy Markdown
Contributor

@dlstucki dlstucki Oct 27, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s

The extra 's' seems like it could be wrong. #Closed

dlstucki
dlstucki previously approved these changes Oct 27, 2021
Copy link
Copy Markdown
Contributor

@dlstucki dlstucki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@dlstucki dlstucki dismissed their stale review October 27, 2021 22:03

revoking review

Copy link
Copy Markdown
Contributor

@dlstucki dlstucki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@bainian12345 bainian12345 merged commit 6939d2d into dev Oct 27, 2021
@bainian12345 bainian12345 deleted the bailiu/azure-identity-tokenprovider branch October 27, 2021 22:07
bainian12345 added a commit that referenced this pull request Jan 20, 2022
* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

Co-authored-by: bailiu <bailiu@microsoft.com>
jbjordan added a commit that referenced this pull request May 21, 2026
* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Enables text send and read operations for WebSocket connections.  (#81)

* text write/read support for websockets

* uri encode request path

* Merge state Add more commits by pushing to the fix/write-read-text branch on
giventocode/azure-relay-java.Revert "uri encode request path"

This reverts commit ee855e0. Review required At least 1 approving review
is required by reviewers with write access. # Please enter the commit message for your changes. Lines
starting All checks have passed# with '#' will be ignored, and an empty message aborts the commit. 1
successful check# @microsoft-cla# On branch fix/write-read-text license/cla — All CLA requirements met.#
Your branch is up to date with 'origin/fix/write-read-text'. Required# Details# Changes to be committed:
Merging is blocked# modified: src/main/java/com/microsoft/azure/relay/HybridHttpConnection.java Merging can
be performed automatically with 1 approving review.# modified:
src/test/java/com/microsoft/azure/relay/HybridConnectionListenerTest.java @giventocode#
 #
 #
Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember,
contributions to this repository should follow its code of conduct.
 ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views. Reviewers No reviews Still in
progress? Assignees No one assigned Labels None yet

* added large message test, pr feedback

* import fixes, and nit

* import fix

* Hybrid HTTP connection handles URL requests with spaces.  (#83)

* encode requests url paths

* PR feedback

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>

* .env, logging, and updated dependencies (#88)

* Replace JSR-356 WebSocket implementation with Jetty (#89)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* Implement keep-alive via WebSocket Ping (#91)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* implement ping

* update interval and fix spacing

* fix whitespace

* formatting

* make ping interval configurable and add test

---------

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>
Co-authored-by: bailiu <bailiu@microsoft.com>
Co-authored-by: Jesus Aguilar <3589801+giventocode@users.noreply.github.com>
Co-authored-by: Todd Venhaus <tvenhaus@gmail.com>
jbjordan added a commit that referenced this pull request May 21, 2026
* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Enables text send and read operations for WebSocket connections.  (#81)

* text write/read support for websockets

* uri encode request path

* Merge state Add more commits by pushing to the fix/write-read-text branch on
giventocode/azure-relay-java.Revert "uri encode request path"

This reverts commit ee855e0. Review required At least 1 approving review
is required by reviewers with write access. # Please enter the commit message for your changes. Lines
starting All checks have passed# with '#' will be ignored, and an empty message aborts the commit. 1
successful check# @microsoft-cla# On branch fix/write-read-text license/cla — All CLA requirements met.#
Your branch is up to date with 'origin/fix/write-read-text'. Required# Details# Changes to be committed:
Merging is blocked# modified: src/main/java/com/microsoft/azure/relay/HybridHttpConnection.java Merging can
be performed automatically with 1 approving review.# modified:
src/test/java/com/microsoft/azure/relay/HybridConnectionListenerTest.java @giventocode#
 #
 #
Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember,
contributions to this repository should follow its code of conduct.
 ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views. Reviewers No reviews Still in
progress? Assignees No one assigned Labels None yet

* added large message test, pr feedback

* import fixes, and nit

* import fix

* Hybrid HTTP connection handles URL requests with spaces.  (#83)

* encode requests url paths

* PR feedback

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>

* .env, logging, and updated dependencies (#88)

* Replace JSR-356 WebSocket implementation with Jetty (#89)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* Implement keep-alive via WebSocket Ping (#91)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* implement ping

* update interval and fix spacing

* fix whitespace

* formatting

* make ping interval configurable and add test

---------

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>
Co-authored-by: bailiu <bailiu@microsoft.com>
Co-authored-by: Jesus Aguilar <3589801+giventocode@users.noreply.github.com>
Co-authored-by: Todd Venhaus <tvenhaus@gmail.com>
jbjordan added a commit that referenced this pull request May 21, 2026
* Merge dev branch into master for version 0.0.4 (#79)

* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

Co-authored-by: bailiu <bailiu@microsoft.com>

* Enables text send and read operations for WebSocket connections.  (#81)

* text write/read support for websockets

* uri encode request path

* Merge state Add more commits by pushing to the fix/write-read-text branch on
giventocode/azure-relay-java.Revert "uri encode request path"

This reverts commit ee855e0. Review required At least 1 approving review
is required by reviewers with write access. # Please enter the commit message for your changes. Lines
starting All checks have passed# with '#' will be ignored, and an empty message aborts the commit. 1
successful check# @microsoft-cla# On branch fix/write-read-text license/cla — All CLA requirements met.#
Your branch is up to date with 'origin/fix/write-read-text'. Required# Details# Changes to be committed:
Merging is blocked# modified: src/main/java/com/microsoft/azure/relay/HybridHttpConnection.java Merging can
be performed automatically with 1 approving review.# modified:
src/test/java/com/microsoft/azure/relay/HybridConnectionListenerTest.java @giventocode#
 #
 #
Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember,
contributions to this repository should follow its code of conduct.
 ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views. Reviewers No reviews Still in
progress? Assignees No one assigned Labels None yet

* added large message test, pr feedback

* import fixes, and nit

* import fix

* Hybrid HTTP connection handles URL requests with spaces.  (#83)

* encode requests url paths

* PR feedback

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>

* Version Update (#90)

Removed Javadoc throws comment and updated version in pom.xml

* Microsoft mandatory file (#85)

Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: Jacob <108886350+jbjordan@users.noreply.github.com>

* Bump json from 20180813 to 20230227 (#92)

Bumps [json](https://github.com/douglascrockford/JSON-java) from 20180813 to 20230227.
- [Release notes](https://github.com/douglascrockford/JSON-java/releases)
- [Changelog](https://github.com/stleary/JSON-java/blob/master/docs/RELEASES.md)
- [Commits](https://github.com/douglascrockford/JSON-java/commits)

---
updated-dependencies:
- dependency-name: org.json:json
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jacob <108886350+jbjordan@users.noreply.github.com>

* Update pom.xml to version 0.0.6 (#93)

* Bump org.json:json from 20230227 to 20231013 (#94)

Bumps [org.json:json](https://github.com/douglascrockford/JSON-java) from 20230227 to 20231013.
- [Release notes](https://github.com/douglascrockford/JSON-java/releases)
- [Changelog](https://github.com/stleary/JSON-java/blob/master/docs/RELEASES.md)
- [Commits](https://github.com/douglascrockford/JSON-java/commits)

---
updated-dependencies:
- dependency-name: org.json:json
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump com.azure:azure-identity from 1.4.0 to 1.12.2 (#95)

Bumps [com.azure:azure-identity](https://github.com/Azure/azure-sdk-for-java) from 1.4.0 to 1.12.2.
- [Release notes](https://github.com/Azure/azure-sdk-for-java/releases)
- [Commits](Azure/azure-sdk-for-java@azure-json_1.4.0...azure-identity_1.12.2)

---
updated-dependencies:
- dependency-name: com.azure:azure-identity
  dependency-version: 1.12.2
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update Util (#96)

* Merge dev into master and resolve conflicts (#97)

Agent-Logs-Url: https://github.com/Azure/azure-relay-java/sessions/570c7374-cd9f-48c1-a806-3192052d742e

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jbjordan <108886350+jbjordan@users.noreply.github.com>

* Revert "Merge dev into master and resolve conflicts (#97)" (#98)

This reverts commit bc10c8c.

* merge dev into main (#100)

* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Enables text send and read operations for WebSocket connections.  (#81)

* text write/read support for websockets

* uri encode request path

* Merge state Add more commits by pushing to the fix/write-read-text branch on
giventocode/azure-relay-java.Revert "uri encode request path"

This reverts commit ee855e0. Review required At least 1 approving review
is required by reviewers with write access. # Please enter the commit message for your changes. Lines
starting All checks have passed# with '#' will be ignored, and an empty message aborts the commit. 1
successful check# @microsoft-cla# On branch fix/write-read-text license/cla — All CLA requirements met.#
Your branch is up to date with 'origin/fix/write-read-text'. Required# Details# Changes to be committed:
Merging is blocked# modified: src/main/java/com/microsoft/azure/relay/HybridHttpConnection.java Merging can
be performed automatically with 1 approving review.# modified:
src/test/java/com/microsoft/azure/relay/HybridConnectionListenerTest.java @giventocode#
 #
 #
Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember,
contributions to this repository should follow its code of conduct.
 ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views. Reviewers No reviews Still in
progress? Assignees No one assigned Labels None yet

* added large message test, pr feedback

* import fixes, and nit

* import fix

* Hybrid HTTP connection handles URL requests with spaces.  (#83)

* encode requests url paths

* PR feedback

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>

* .env, logging, and updated dependencies (#88)

* Replace JSR-356 WebSocket implementation with Jetty (#89)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* Implement keep-alive via WebSocket Ping (#91)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* implement ping

* update interval and fix spacing

* fix whitespace

* formatting

* make ping interval configurable and add test

---------

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>
Co-authored-by: bailiu <bailiu@microsoft.com>
Co-authored-by: Jesus Aguilar <3589801+giventocode@users.noreply.github.com>
Co-authored-by: Todd Venhaus <tvenhaus@gmail.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Bainian Liu <bainian12345@hotmail.com>
Co-authored-by: bailiu <bailiu@microsoft.com>
Co-authored-by: Jesus Aguilar <3589801+giventocode@users.noreply.github.com>
Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Todd Venhaus <tvenhaus@gmail.com>
jbjordan added a commit that referenced this pull request May 21, 2026
* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Enables text send and read operations for WebSocket connections.  (#81)

* text write/read support for websockets

* uri encode request path

* Merge state Add more commits by pushing to the fix/write-read-text branch on
giventocode/azure-relay-java.Revert "uri encode request path"

This reverts commit ee855e0. Review required At least 1 approving review
is required by reviewers with write access. # Please enter the commit message for your changes. Lines
starting All checks have passed# with '#' will be ignored, and an empty message aborts the commit. 1
successful check# @microsoft-cla# On branch fix/write-read-text license/cla — All CLA requirements met.#
Your branch is up to date with 'origin/fix/write-read-text'. Required# Details# Changes to be committed:
Merging is blocked# modified: src/main/java/com/microsoft/azure/relay/HybridHttpConnection.java Merging can
be performed automatically with 1 approving review.# modified:
src/test/java/com/microsoft/azure/relay/HybridConnectionListenerTest.java @giventocode#
 #
 #
Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember,
contributions to this repository should follow its code of conduct.
 ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views. Reviewers No reviews Still in
progress? Assignees No one assigned Labels None yet

* added large message test, pr feedback

* import fixes, and nit

* import fix

* Hybrid HTTP connection handles URL requests with spaces.  (#83)

* encode requests url paths

* PR feedback

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>

* .env, logging, and updated dependencies (#88)

* Replace JSR-356 WebSocket implementation with Jetty (#89)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* Implement keep-alive via WebSocket Ping (#91)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* implement ping

* update interval and fix spacing

* fix whitespace

* formatting

* make ping interval configurable and add test

* Rebase dev (#104)

* Merge dev branch into master for version 0.0.4 (#79)

* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

Co-authored-by: bailiu <bailiu@microsoft.com>

* Enables text send and read operations for WebSocket connections.  (#81)

* text write/read support for websockets

* uri encode request path

* Merge state Add more commits by pushing to the fix/write-read-text branch on
giventocode/azure-relay-java.Revert "uri encode request path"

This reverts commit ee855e0. Review required At least 1 approving review
is required by reviewers with write access. # Please enter the commit message for your changes. Lines
starting All checks have passed# with '#' will be ignored, and an empty message aborts the commit. 1
successful check# @microsoft-cla# On branch fix/write-read-text license/cla — All CLA requirements met.#
Your branch is up to date with 'origin/fix/write-read-text'. Required# Details# Changes to be committed:
Merging is blocked# modified: src/main/java/com/microsoft/azure/relay/HybridHttpConnection.java Merging can
be performed automatically with 1 approving review.# modified:
src/test/java/com/microsoft/azure/relay/HybridConnectionListenerTest.java @giventocode#
 #
 #
Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember,
contributions to this repository should follow its code of conduct.
 ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views. Reviewers No reviews Still in
progress? Assignees No one assigned Labels None yet

* added large message test, pr feedback

* import fixes, and nit

* import fix

* Hybrid HTTP connection handles URL requests with spaces.  (#83)

* encode requests url paths

* PR feedback

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>

* Version Update (#90)

Removed Javadoc throws comment and updated version in pom.xml

* Microsoft mandatory file (#85)

Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: Jacob <108886350+jbjordan@users.noreply.github.com>

* Bump json from 20180813 to 20230227 (#92)

Bumps [json](https://github.com/douglascrockford/JSON-java) from 20180813 to 20230227.
- [Release notes](https://github.com/douglascrockford/JSON-java/releases)
- [Changelog](https://github.com/stleary/JSON-java/blob/master/docs/RELEASES.md)
- [Commits](https://github.com/douglascrockford/JSON-java/commits)

---
updated-dependencies:
- dependency-name: org.json:json
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jacob <108886350+jbjordan@users.noreply.github.com>

* Update pom.xml to version 0.0.6 (#93)

* Bump org.json:json from 20230227 to 20231013 (#94)

Bumps [org.json:json](https://github.com/douglascrockford/JSON-java) from 20230227 to 20231013.
- [Release notes](https://github.com/douglascrockford/JSON-java/releases)
- [Changelog](https://github.com/stleary/JSON-java/blob/master/docs/RELEASES.md)
- [Commits](https://github.com/douglascrockford/JSON-java/commits)

---
updated-dependencies:
- dependency-name: org.json:json
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump com.azure:azure-identity from 1.4.0 to 1.12.2 (#95)

Bumps [com.azure:azure-identity](https://github.com/Azure/azure-sdk-for-java) from 1.4.0 to 1.12.2.
- [Release notes](https://github.com/Azure/azure-sdk-for-java/releases)
- [Commits](Azure/azure-sdk-for-java@azure-json_1.4.0...azure-identity_1.12.2)

---
updated-dependencies:
- dependency-name: com.azure:azure-identity
  dependency-version: 1.12.2
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update Util (#96)

* Merge dev into master and resolve conflicts (#97)

Agent-Logs-Url: https://github.com/Azure/azure-relay-java/sessions/570c7374-cd9f-48c1-a806-3192052d742e

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jbjordan <108886350+jbjordan@users.noreply.github.com>

* Revert "Merge dev into master and resolve conflicts (#97)" (#98)

This reverts commit bc10c8c.

* merge dev into main (#100)

* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Enables text send and read operations for WebSocket connections.  (#81)

* text write/read support for websockets

* uri encode request path

* Merge state Add more commits by pushing to the fix/write-read-text branch on
giventocode/azure-relay-java.Revert "uri encode request path"

This reverts commit ee855e0. Review required At least 1 approving review
is required by reviewers with write access. # Please enter the commit message for your changes. Lines
starting All checks have passed# with '#' will be ignored, and an empty message aborts the commit. 1
successful check# @microsoft-cla# On branch fix/write-read-text license/cla — All CLA requirements met.#
Your branch is up to date with 'origin/fix/write-read-text'. Required# Details# Changes to be committed:
Merging is blocked# modified: src/main/java/com/microsoft/azure/relay/HybridHttpConnection.java Merging can
be performed automatically with 1 approving review.# modified:
src/test/java/com/microsoft/azure/relay/HybridConnectionListenerTest.java @giventocode#
 #
 #
Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember,
contributions to this repository should follow its code of conduct.
 ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views. Reviewers No reviews Still in
progress? Assignees No one assigned Labels None yet

* added large message test, pr feedback

* import fixes, and nit

* import fix

* Hybrid HTTP connection handles URL requests with spaces.  (#83)

* encode requests url paths

* PR feedback

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>

* .env, logging, and updated dependencies (#88)

* Replace JSR-356 WebSocket implementation with Jetty (#89)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* Implement keep-alive via WebSocket Ping (#91)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* implement ping

* update interval and fix spacing

* fix whitespace

* formatting

* make ping interval configurable and add test

---------

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>
Co-authored-by: bailiu <bailiu@microsoft.com>
Co-authored-by: Jesus Aguilar <3589801+giventocode@users.noreply.github.com>
Co-authored-by: Todd Venhaus <tvenhaus@gmail.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Bainian Liu <bainian12345@hotmail.com>
Co-authored-by: bailiu <bailiu@microsoft.com>
Co-authored-by: Jesus Aguilar <3589801+giventocode@users.noreply.github.com>
Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Todd Venhaus <tvenhaus@gmail.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Bainian Liu <bainian12345@hotmail.com>
Co-authored-by: bailiu <bailiu@microsoft.com>
Co-authored-by: Jesus Aguilar <3589801+giventocode@users.noreply.github.com>
Co-authored-by: Todd Venhaus <tvenhaus@gmail.com>
Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
jbjordan added a commit that referenced this pull request May 21, 2026
* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Enables text send and read operations for WebSocket connections.  (#81)

* text write/read support for websockets

* uri encode request path

* Merge state Add more commits by pushing to the fix/write-read-text branch on
giventocode/azure-relay-java.Revert "uri encode request path"

This reverts commit ee855e0. Review required At least 1 approving review
is required by reviewers with write access. # Please enter the commit message for your changes. Lines
starting All checks have passed# with '#' will be ignored, and an empty message aborts the commit. 1
successful check# @microsoft-cla# On branch fix/write-read-text license/cla — All CLA requirements met.#
Your branch is up to date with 'origin/fix/write-read-text'. Required# Details# Changes to be committed:
Merging is blocked# modified: src/main/java/com/microsoft/azure/relay/HybridHttpConnection.java Merging can
be performed automatically with 1 approving review.# modified:
src/test/java/com/microsoft/azure/relay/HybridConnectionListenerTest.java @giventocode#
 #
 #
Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember,
contributions to this repository should follow its code of conduct.
 ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views. Reviewers No reviews Still in
progress? Assignees No one assigned Labels None yet

* added large message test, pr feedback

* import fixes, and nit

* import fix

* Hybrid HTTP connection handles URL requests with spaces.  (#83)

* encode requests url paths

* PR feedback

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>

* .env, logging, and updated dependencies (#88)

* Replace JSR-356 WebSocket implementation with Jetty (#89)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* Implement keep-alive via WebSocket Ping (#91)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* implement ping

* update interval and fix spacing

* fix whitespace

* formatting

* make ping interval configurable and add test

* Rebase dev (#104)

* Merge dev branch into master for version 0.0.4 (#79)

* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

Co-authored-by: bailiu <bailiu@microsoft.com>

* Enables text send and read operations for WebSocket connections.  (#81)

* text write/read support for websockets

* uri encode request path

* Merge state Add more commits by pushing to the fix/write-read-text branch on
giventocode/azure-relay-java.Revert "uri encode request path"

This reverts commit ee855e0. Review required At least 1 approving review
is required by reviewers with write access. # Please enter the commit message for your changes. Lines
starting All checks have passed# with '#' will be ignored, and an empty message aborts the commit. 1
successful check# @microsoft-cla# On branch fix/write-read-text license/cla — All CLA requirements met.#
Your branch is up to date with 'origin/fix/write-read-text'. Required# Details# Changes to be committed:
Merging is blocked# modified: src/main/java/com/microsoft/azure/relay/HybridHttpConnection.java Merging can
be performed automatically with 1 approving review.# modified:
src/test/java/com/microsoft/azure/relay/HybridConnectionListenerTest.java @giventocode#
 #
 #
Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember,
contributions to this repository should follow its code of conduct.
 ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views. Reviewers No reviews Still in
progress? Assignees No one assigned Labels None yet

* added large message test, pr feedback

* import fixes, and nit

* import fix

* Hybrid HTTP connection handles URL requests with spaces.  (#83)

* encode requests url paths

* PR feedback

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>

* Version Update (#90)

Removed Javadoc throws comment and updated version in pom.xml

* Microsoft mandatory file (#85)

Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: Jacob <108886350+jbjordan@users.noreply.github.com>

* Bump json from 20180813 to 20230227 (#92)

Bumps [json](https://github.com/douglascrockford/JSON-java) from 20180813 to 20230227.
- [Release notes](https://github.com/douglascrockford/JSON-java/releases)
- [Changelog](https://github.com/stleary/JSON-java/blob/master/docs/RELEASES.md)
- [Commits](https://github.com/douglascrockford/JSON-java/commits)

---
updated-dependencies:
- dependency-name: org.json:json
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jacob <108886350+jbjordan@users.noreply.github.com>

* Update pom.xml to version 0.0.6 (#93)

* Bump org.json:json from 20230227 to 20231013 (#94)

Bumps [org.json:json](https://github.com/douglascrockford/JSON-java) from 20230227 to 20231013.
- [Release notes](https://github.com/douglascrockford/JSON-java/releases)
- [Changelog](https://github.com/stleary/JSON-java/blob/master/docs/RELEASES.md)
- [Commits](https://github.com/douglascrockford/JSON-java/commits)

---
updated-dependencies:
- dependency-name: org.json:json
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump com.azure:azure-identity from 1.4.0 to 1.12.2 (#95)

Bumps [com.azure:azure-identity](https://github.com/Azure/azure-sdk-for-java) from 1.4.0 to 1.12.2.
- [Release notes](https://github.com/Azure/azure-sdk-for-java/releases)
- [Commits](Azure/azure-sdk-for-java@azure-json_1.4.0...azure-identity_1.12.2)

---
updated-dependencies:
- dependency-name: com.azure:azure-identity
  dependency-version: 1.12.2
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update Util (#96)

* Merge dev into master and resolve conflicts (#97)

Agent-Logs-Url: https://github.com/Azure/azure-relay-java/sessions/570c7374-cd9f-48c1-a806-3192052d742e

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jbjordan <108886350+jbjordan@users.noreply.github.com>

* Revert "Merge dev into master and resolve conflicts (#97)" (#98)

This reverts commit bc10c8c.

* merge dev into main (#100)

* improve synchronization of fields (#68)

Co-authored-by: bailiu <bailiu@microsoft.com>

* make status description for HTTP response be JSON safe (#69)

* make status description for HTTP response be JSON safe

* assert response description

Co-authored-by: bailiu <bailiu@microsoft.com>

* allow empty URI query param (#67)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Add new API to allow authentication with Azure Identity (#76)

* cleanup unnecessary pom configurations

* Enable authentication using Azure Identity

* add copyright headers

Co-authored-by: bailiu <bailiu@microsoft.com>

* Updating dependency according to github issues (#77)

Co-authored-by: bailiu <bailiu@microsoft.com>

* update version to 0.0.4 (#78)

Co-authored-by: bailiu <bailiu@microsoft.com>

* Enables text send and read operations for WebSocket connections.  (#81)

* text write/read support for websockets

* uri encode request path

* Merge state Add more commits by pushing to the fix/write-read-text branch on
giventocode/azure-relay-java.Revert "uri encode request path"

This reverts commit ee855e0. Review required At least 1 approving review
is required by reviewers with write access. # Please enter the commit message for your changes. Lines
starting All checks have passed# with '#' will be ignored, and an empty message aborts the commit. 1
successful check# @microsoft-cla# On branch fix/write-read-text license/cla — All CLA requirements met.#
Your branch is up to date with 'origin/fix/write-read-text'. Required# Details# Changes to be committed:
Merging is blocked# modified: src/main/java/com/microsoft/azure/relay/HybridHttpConnection.java Merging can
be performed automatically with 1 approving review.# modified:
src/test/java/com/microsoft/azure/relay/HybridConnectionListenerTest.java @giventocode#
 #
 #
Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember,
contributions to this repository should follow its code of conduct.
 ProTip! Add .patch or .diff to the end of URLs for Git’s plaintext views. Reviewers No reviews Still in
progress? Assignees No one assigned Labels None yet

* added large message test, pr feedback

* import fixes, and nit

* import fix

* Hybrid HTTP connection handles URL requests with spaces.  (#83)

* encode requests url paths

* PR feedback

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>

* .env, logging, and updated dependencies (#88)

* Replace JSR-356 WebSocket implementation with Jetty (#89)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* Implement keep-alive via WebSocket Ping (#91)

* .env, logging, and updated dependencies

* replace javax.websockets with jetty websockets

* implement ping

* update interval and fix spacing

* fix whitespace

* formatting

* make ping interval configurable and add test

---------

Co-authored-by: Bainian Liu <bainian12345@hotmail.com>
Co-authored-by: bailiu <bailiu@microsoft.com>
Co-authored-by: Jesus Aguilar <3589801+giventocode@users.noreply.github.com>
Co-authored-by: Todd Venhaus <tvenhaus@gmail.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Bainian Liu <bainian12345@hotmail.com>
Co-authored-by: bailiu <bailiu@microsoft.com>
Co-authored-by: Jesus Aguilar <3589801+giventocode@users.noreply.github.com>
Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Todd Venhaus <tvenhaus@gmail.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Bainian Liu <bainian12345@hotmail.com>
Co-authored-by: bailiu <bailiu@microsoft.com>
Co-authored-by: Jesus Aguilar <3589801+giventocode@users.noreply.github.com>
Co-authored-by: Todd Venhaus <tvenhaus@gmail.com>
Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants