-
Notifications
You must be signed in to change notification settings - Fork 156
Convert most AMQP types to read-only; Significant API surface cleanup. #4640
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
LarryOsterman
merged 38 commits into
Azure:main
from
LarryOsterman:larryo/readonlyconnectionandsession
May 26, 2023
Merged
Changes from 28 commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
b749504
Prep for readonly connection
LarryOsterman 70ecf68
Added options for Connection and Session to enable read-only configur…
LarryOsterman 9d51ba5
Read-only connection and session objects; use message source and targ…
LarryOsterman 2d2d7af
Enabled read-only session; added tests for incorrect AMQPS port
LarryOsterman 15cd3ab
Cleanup of objects
LarryOsterman d9fe28e
gcc9 fixes
LarryOsterman 4724a20
gcc9 fixes 2
LarryOsterman c405933
gcc9 fixes 3
LarryOsterman f434991
gcc9 fixes 3
LarryOsterman 3661f19
gcc9 fixes 4
LarryOsterman a972875
gcc9 fixes 5
LarryOsterman ec67c2a
clang-format; cspell
LarryOsterman 08b51d7
doxygen
LarryOsterman 11b8f64
doxygen 2
LarryOsterman 3a6384b
Fixed line accidentally removed
LarryOsterman 23b81e8
Sas credentials are azure credentials; removed sas credential option …
LarryOsterman 68d2bbb
Moved credentials to session; reworked how audience is calculated for…
LarryOsterman fe0e7dd
clang-format
LarryOsterman da97d9c
clang fix
LarryOsterman ee17b57
CI fixes
LarryOsterman b72743b
Mac doesn't talk sockets
LarryOsterman 2f9ef28
Added test-resources.json for samples to ease deployment
LarryOsterman 28028c5
Updated test-resources.json to grant the test user the ability to rea…
LarryOsterman e723b99
Update sdk/core/azure-core-amqp/README.md
LarryOsterman 8e1bf8b
Added token reader sample; flushed out eventhubs properties sample, a…
LarryOsterman 72ef244
Merge branch 'larryo/readonlyconnectionandsession' of https://github.…
LarryOsterman 875c890
PUll request feedback
LarryOsterman 604d4ce
More PR feedback
LarryOsterman e956dc6
PR feedback
LarryOsterman 6cd6401
Moved AMQP samples to internal
LarryOsterman c2ced30
Moved samples to use GetEnvHelper
LarryOsterman e535e60
PR feedback; Don't poll when connection is in the 'End' state
LarryOsterman 850d0e6
clang-format
LarryOsterman 35c1706
Improved diagnostics for cbs open to help detect CBS errors in tests
LarryOsterman 1110c0c
More pull request feedback - removed lots of fully qualified namespaces
LarryOsterman 0c73559
more pull request feedback
LarryOsterman 8c8277e
clang-format
LarryOsterman 0759b79
Prevent creation of Messaging static-only class
LarryOsterman File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.