EG API updates #18274
Merged
EG API updates #18274
Conversation
This file contains 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
JoshLove-msft
requested review from
ahamad-MS,
jfggdl,
Kishp01,
KrzysztofCwalina and
tg-msft
as code owners
ghost
added
the
JoshLove-msft
commented
Jan 29, 2021
| public EventGridPublisherClient(System.Uri endpoint, Azure.AzureSasCredential credential) { } | ||
| public EventGridPublisherClient(System.Uri endpoint, Azure.AzureSasCredential credential, Azure.Messaging.EventGrid.EventGridPublisherClientOptions options) { } | ||
| public static string BuildSharedAccessSignature(System.Uri endpoint, System.DateTimeOffset expirationUtc, Azure.AzureKeyCredential key, Azure.Messaging.EventGrid.EventGridPublisherClientOptions.ServiceVersion apiVersion = Azure.Messaging.EventGrid.EventGridPublisherClientOptions.ServiceVersion.V2018_01_01) { throw null; } | ||
| public EventGridPublisherClient(System.Uri endpoint, Azure.AzureSasCredential credential, Azure.Messaging.EventGrid.EventGridPublisherClientOptions options = null) { } |
There was a problem hiding this comment.
Collapsed this into a single ctor with default options since this is a more advanced ctor.
JoshLove-msft
commented
Jan 29, 2021
| uriBuilder.Reset(Endpoint); | ||
| uriBuilder.AppendQuery("api-version", ApiVersion.GetVersionString(), true); | ||
| string encodedResource = HttpUtility.UrlEncode(uriBuilder.ToString()); | ||
| var encodedExpirationUtc = HttpUtility.UrlEncode(ExpiresOn.ToString(CultureInfo.InvariantCulture)); |
There was a problem hiding this comment.
This was previously using "en-us" for the culture as that is what was referenced in https://docs.microsoft.com/en-us/azure/event-grid/security-authenticate-publishing-clients#generate-sas-token-programmatically
But InvariantCulture seems to work fine.
There was a problem hiding this comment.
Slight preference for keeping en-US since that's what was documented by the service, but I agree in practice there is unlikely to be a difference between the two cultures in practice.
There was a problem hiding this comment.
Fair enough, will revert.
JoshLove-msft
commented
Jan 29, 2021
| @@ -28,7 +28,7 @@ public partial class CloudEvent | |||
| } | |||
| public partial class EventGridEvent | |||
| { | |||
| public EventGridEvent(object data, string subject, string eventType, string dataVersion, System.Type dataSerializationType = null) { } | |||
| public EventGridEvent(string subject, string eventType, string dataVersion, object data, System.Type dataSerializationType = null) { } | |||
There was a problem hiding this comment.
moved data to be the last required param to be consistent with CloudEvent and to make it easier if inlining a new data object. The one difference is that in CloudEvent data is the first of the data-related params because the other params are optional. In EGEvent dataVersion is required.
Perhaps it would be better to keep the relative ordering consistent.
There was a problem hiding this comment.
I still like having it last in the list of required parameters for ergonomics, even though there's a slight inconsistency between Cloud Events and Event Grid Events
ellismg
approved these changes
Jan 29, 2021
I just submitted last night so this was fast! |
|
/azp run net - eventgrid - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
markcowl
pushed a commit
that referenced
this pull request
Feb 12, 2021
* [Service Bus Cient] Client Mockability Fix (#18254) The focus of these changes is to virtualize read-only properties that cannot be set as part of the constructor to enable mocking them during testing. * Adjust build warning settings * Sync eng/common directory with azure-sdk-tools repository (#18282) * Enable testing of public_exponent against MHSM (#18283) Resolves #16789 * Add CreateEcKeyOptions class (#18272) Resolves #16853 * Log more information for canceled CertificationOperations (#18289) Relates to #17718 * EG API updates (#18274) * EG API updates * webjob tests * Use optional serializer param in GetData<T> methods; revert culture in SAS * Additional operations and test cases for Azure Quantum SDK (#18239) Co-authored-by: Victor XField <[email protected]> * Add SyncAsyncEventHandler (#18170) * Add support for create, update, and delete for custom role definitions (#18227) * Implement create, update, get, and delete for Role Definitions * [FR] Modify `Pages` type (#18260) Changed the type for `IEnumerable` to `IList` as this is a property in an input-only object. Followed what other libraries like Search and KV do. * Quota API GA version SDK updates. (#17965) * Updates for GA version - <AzureApiTag>Capacity_2020-10-01-preview;Capacity_2020-10-25;</AzureApiTag> * Updating release notes. * Updating version to -Preview - 1.15.0-preview * Updated Package release notes. * Removing -Preview from changelog.md to fix CI build error. * Adding -Preview to Package version. * Updating recorded Tests. * Updating Tests. * Uprating the version to - <Version>1.15.0-preview</Version> in csproj file. * Updating Tests. * Updating commit ID. * Removing model files not nrequired. * Move global variable to the pipeline level (#18288) To ensure we always include the global variables in all our pipeline jobs and stages we should import the variables at the highest pipeline level to ensure we don't miss setting these variables in any jobs. * Sync eng/common directory with azure-sdk-tools for PR 1345 (#18287) * Enforce API approval status for GA and include SDK type in package properties * Fix per review comment * Fixes as per review comments to avoid changing constructor signature * Fixes as per review comments to handle track 1 packages * Fix yaml format error Co-authored-by: praveenkuttappan <[email protected]> * Initial commit * Temporarily re-enable .NET 5 SDK install step (#18298) * Fix Waqar's comment. * [Storage] Queues - Failed message handler (#17001) In this PR: - Ability to provide a handler to capture queue messages that are in invalid format. Bonus: - Ability to get parent queue service (like we did in #16437 ) Pending dependency: #18170 * [FR] Language and Locale as extensible enums (#18271) * language and locale as extensible enums * Import global variables all mgmt jobs * [Identity] Preparing for Azure.Identity 1.4.0-beta.2 release (#18281) * upgrading msal dependency * update changelog * updating test recordings * Ignore sample tests (#18292) The sample tests weren't intended to be run in the CI, but were instead useful to demonstrate how you might perform the different methods of authentication. That's actually done in the README now, so i'm just going to remove it since I can't make the build happy with keeping them around. * Increment version for identity releases (#18301) Increment package version after release of Azure.Identity * Updated with success cases. * 1ES pools update for release pipeline. (#18267) * Sync eng/common directory with azure-sdk-tools for PR 1351 (#18314) * tweaks to New-TestResources readme * [EngSys] Fixing CODEOWNERS Typo (#18316) The focus of these changes is to fix a typo in label application for the Digital Twins service. The ownership of Service Bus has also been tweaked to reflect a change in areas of responsibility. * Update AutoRest C# version (#18249) Co-authored-by: Shivangi Reja <[email protected]> * acr: update change log for releasing (#18303) * Update the readme with more information about thread safety and other… (#18290) * Update the readme with more information about thread safety and other concepts. * Update sdk/keyvault/Azure.Security.KeyVault.Administration/README.md Co-authored-by: Mariana Rios Flores <[email protected]> * Update sdk/keyvault/Azure.Security.KeyVault.Administration/README.md Co-authored-by: Heath Stewart <[email protected]> * Address PR feedback. Co-authored-by: Mariana Rios Flores <[email protected]> Co-authored-by: Heath Stewart <[email protected]> * Made Storage Client CanGenerateSasUri properties mockable (#18277) * Add APIView KV variable group to prepare pipeline (#18304) * Adding more test cases for Azure Quantum Mgmt SDK (#18247) * Improve Az Login logic for recording auth * Adding remaining test cases * Update webjobs queue extension CHANGELOG.md for base64 (#18325) Provided a sample host.json to make it easier for functions uses to take advantage of the ability to opt-out of base64 encoding. * Update samples (#18293) * [MetricsAdvisor] Fixing failing test (#18321) * Azure Quantum SDK Readme (#18296) Authored-by: matusthemostlygreat <[email protected]> * Verify API status check changes (#18176) Draft PR to verify common changes. * Update CHANGELOG.md (#18333) * Fix sample (#18332) * Fix sample * Fix * Update AutoRest C# version (#18331) * Regenerate baseline because last one break. (#18270) * Updating CODEOWMNERS entry for Azure.Identity (#18336) * fix (#18327) * Fix DateTime bug (#18340) Co-authored-by: Chidozie Ononiwu <[email protected]> * [Communciation] - Remove nullable reftypes from the public surface of the Identity and Common package (#18334) * Revert "Temporarily re-enable .NET 5 SDK install step (#18298)" (#18324) This reverts commit 21ebca7. * Base64 encoding remarks. (#18345) * Document that DeletedSecret.Value is always null (#18300) Resolves #17741 To note, I also checked if/how this applies to DeletedKey and DeletedCertificate. When retrieved individually, the public key portion of the JWK is retrieved and both the SecretId and KeyId on DeletedCertificate are present as well. * Update IE E2E. * Normalize the package name for Doc.Ms readme (#18262) * Sync eng/common directory with azure-sdk-tools for PR 1353 (#18348) * Use a helper function to normalize the package name for readme * Get the readme name from artifact-metadata-parsing * Change to the readme * Update eng/common/scripts/artifact-metadata-parsing.ps1 Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Sima Zhu <[email protected]> Co-authored-by: Sima Zhu <[email protected]> Co-authored-by: Wes Haggard <[email protected]> * [MetricsAdvisor] Made constructor of DataFeed parameterless (#18319) * Fixed bug when setting Viewers and Administrators during Data Feed creation. * Made DataFeed-related collections not settable. * [TA] Healthcare Design update (#18200) * [Storage] Badly encoded handler in Webjobs (#18326) * wip * export api. * that works. * handle peeked messages * api * post merge. * propagate queueclient. * fire and forget callback. * tweaks. * re-record. * hack core temporarily. * use event hander from core. * revert test change. * remove direct core reference from test package. * that won't be necessary. * more tests. * readme. * whitespace. * change options creation. * some pr feedback. * api * get parent queue service. * can handle bad message. * blob trigger. * post-merge. * post-merge. * Increment version for quantum releases (#18337) Increment package version after release of Azure.Quantum.Jobs * fix typo (#18335) * Extending TokenRequestContext to accept additional Claims (#18236) * Extending TokenRequestContext to accept additional Claims * updating api definitions * fix ctor ambiguity * [Event Hubs Client] Formatting Pass (#18356) The purpose of these changes is to format code to apply project conventions for consistency and in some cases, update member names and documentation to better convey context. **Note:** These changes are intended to be superficial; no change to existing behavior should have taken place. * Remove IDisposable from TokenCache (#18346) * Remove IDisposable from TokenCache * Remove unused autores.md file (#18294) * Remove unused autores.md file * Update generate.ps1 * Generate 2020-03-01 2020-07-01-preview policy operations (#18194) * Generate 2020-03-01 2020-07-01-preview policy op * generated policy 2020-09-01 * [Storage] Webjobs extension sample apps. (#18358) * nest sample tests? * queues sample app. * nesting. * new link. * blobs sample. * fix build. * fix build. * does this fix the build ? * Mixed Reality Authentication: Updated the MR STS spec file (#18341) This change updated the library to consume the latest version of spec file, which changes the account identifier to be a `Guid` rather than a `string`. The service expects a `Guid` or `UUID`, so it makes sense to be specific. * Refactor TestEnvVar (#18364) Improve `TestEnvVar` to handle consistent cleaning and stashing of existing environment variables. * Adding PR template for net repo (#18357) * Adding PR template * CR feedback * EG API Updates (#18369) * Update API based on review feedback * Update samples * fix * Update wording * Fix sample * CloudServices 43-preview release update (#18372) * hopefully fixed live tests (#18376) * Remove nullable refs sms (#18370) * Remove nullable ref-types from sms sdk * Code review changes - add a basic constructor (with connection string only) for sms client * code review changes * fix for build check failures * Bringing the feature branch for chat API version 2020-11-01-preview3 to master (#18388) Co-authored-by: Camilo Ramirez <[email protected]> * [DataFactory]Merge new features into master (#18380) * Sync eng/common directory with azure-sdk-tools for PR 1364 (#18394) * Support sub directory for artifact location * Fix as per review comment to avoid artifact sub path * Package property file is not created for track1 and management pacakges. * Fix property name case * Fix as per review comments * Fix as per review comments * Use Config File param value as is * Change to make path joineasily readable Co-authored-by: praveenkuttappan <[email protected]> * [Core.Experimental] Add DynamicContent/DynamicRequest/DynamicResponse for LLC prototype (#18323) - Originally from https://github.com/ellismg/AnomalyDetectorLowLevelClient/ * Sync eng/common directory with azure-sdk-tools for PR 1365 (#18396) * Updated cosmos emulator yml script to remove the existing installation, install latest version and run emulator with latest version * Added default emulator start arguments if none provided * Updated default start params * Updated default start params in PS script * Updated default start params in PS script Co-authored-by: Kushagra Thapar <[email protected]> * [Communication] - Simplify identifier json models (#18389) * [Communication] - Simplify identifier json models * Add validation for multiple properties being present * [Event Hubs Client] Move Blob Checkpoint Store to Shared (#18395) The focus of these changes is to complete migration of the Blobs Checkpoint Store from the `Azure.Messaging.EventHubs.Processor` project to the `Azure.Messaging.EventHubs.Shared` project. To facilitate sharing the checkpoint store between the processor and the Functions extensions, the implementation for the store was moved into the shared project, though its tests remained in the processor. In order to follow the pattern of keeping locality between shared code items and their tests, the checkpoint store tests, both unit and live, have been migrated to the shared testing project. In support of this, some of the live testing infrastructure specific to managing storage resources has also been migrated into the shared project as part of the `BlobTesting` category of shared items. Some refactoring and reformatting of the storge tests has also been performed to improve consistency with other Event Hubs code, trim dead areas, and improve readability. * Delete CloudService tests from the stable release (#18390) * Delete CloudServiceExtensionTests.cs * Delete CloudServiceOperationTests.cs * Delete CloudServiceRoleInstanceTests.cs * Delete CloudServiceScenarioTests.cs * Delete CloudServiceTestsBase.cs * Delete CloudServiceUpdateTests.cs * Delete MultiRole_CreateUpdateGetAndDeleteWithExtension_WorkerAndWebRole.json * Delete MultiRole_CreateUpdateGetAndDeleteWithExtension_WorkerAndWebRole_MultiRoleExtension.json * Delete Test_Create_PowerOff_Start_CloudServiceOperation.json * Delete Test_ListCloudServicesOperation.json * Delete Test_CloudServiceRoleInstanceOperations.json * Delete TestCloudServiceScenarioOperations.json * Delete TestCloudServiceScenarioOperations_DeleteCloudService.json * Delete TestCloudServiceScenarioOperations_ExtensionProfile.json * Delete TestCloudServiceScenarioOperations_InstanceView.json * Delete TestCloudServiceOperations_ScaleIn_ScaleOut.json * Delete TestCloudServiceUpdateOperations.json * Update AzSdk.RP.props * Update AzSdk.RP.props remove containerService * Delete ServiceConfigurationHelpers.cs * ADT: Fix the issue with non-default object serializers in Query APIs (#18379) * Fix the bug with non-default ObjectSerializer in QueryClient * [Communication] - Follow standards for identity client options version (#18400) * [TA] PageableOperation implementation (#18351) * Sync eng/common directory with azure-sdk-tools for PR 1366 (#18398) * Scan all duplicate artifacts which set hide to true * Update eng/common/docgeneration/Generate-DocIndex.ps1 Co-authored-by: Wes Haggard <[email protected]> * Update Generate-DocIndex.ps1 Co-authored-by: Sima Zhu <[email protected]> Co-authored-by: Sima Zhu <[email protected]> Co-authored-by: Wes Haggard <[email protected]> * [Event Hubs Client] Producer Dead Code Removal (#18424) The focus of these changes is to trim the dead code paths for publishing a single event from the producer client and assocaited tests. This overload existed in early pre-GA betas but has been determined to be unlikely to return to the public API in the future. * Bump digital twins version for 20201-02-03 (#18403) * Update xml documentation for TokenRequestContext.Claims (#18367) * Update xml documentation for TokenRequestContext.Claims * fix cref to href * Increment version for digitaltwins releases (#18430) Increment package version after release of Azure.DigitalTwins.Core * uncomment readme links. (#18429) * [MetricsAdvisor] Made constructor of AnomalyDetectionConfiguration parameterless (#18393) * Update PR with inbound endpoint changes. * renames, modify types, test cleanup (#18421) * Use Utf8JsonWriter to write JsonElement to stream instead of using th… (#18436) * bump webjobs. (#18428) * Remove extra dependency (#18438) * Update AutoRest C# version (#18338) * Implement BearerTokenChallengeAuthenticationPolicy (#18368) Handles bearer token authorization challenges and is extensible for sub-class implementers. closes #15650 * [IoTCentral] resourceType became camel case, added industry, locations field to appTemplate API, added origin, properties field to operations API (#18415) * include camel case, apptemplateName changes * add industry and locations to appTemplate api * update unit test * [TA] Implement Pageable actions (#18371) * Migrate SB extensions to new SDK (#18330) * Migrate SB extensions to new SDK * Fixes * API * Remove ExceptionHandler API * PR fb * Fix test * add back cleanupentity * Update SyncAsyncEventArgs APIs (#18437) * [Event Hubs Client] Storage Implementation Feedback (#18439) The focus of these changes is to consider feedback provided by an internal review of the blob storage manager by one of the developers responsible for the Azure Blob Storage client library. Many of the larger and more critical items have already been used for adjustments; these tweaks address the majority of the remaining minor items. * Add default constructor to CertificatePolicy for import (#18443) Resolves #16217 * Generate CDN SDK for 2020-09-01 AFDX support (#18218) * Generate 2020-09 version code * add cdn_resource-manager.txt, update ApiVersion modifier * update passed records * update test record, fix failed case * Add AFDX source test and records * Update test Records, Update test utilities * Add secret test and records * Update with swagger * Update with new swagger * Use latest swagger * Add test case and records * Update test cases and records * Update the package version Co-authored-by: Chenglong Liu <[email protected]> * Code style and syntax clean up (#18445) * Code style and syntax clean up * regenrate the code. Co-authored-by: Azad Abbasi <[email protected]> * Revert "[TestFramework] Downgrade Azure.Identity for maximum compat (#18153)" (#18450) - Azure.Identity 1.0.0 does not work on sovereign cloud which is required for our functional tests - Reverting until we can find a better solution for both perf and functional tests - This reverts commit 0f409d2. * Merge always updates for batch operations (#18446) * Mixed Reality Authentication: This change builds and tests the shared code (#18397) This change does the following: - Added a new Share.Azure.MixedReality.Authentication.Tests project to test the shared code. I did find 1 bug that presented when compiling with .NET 5.0 due to better nullable annotations. I had to pull the shared code in as source because I couldn't get `InternalsVisisbleTo` to work in this repo. - This change adds a project file to build the Mixed Reality Authentication shared code to ensure it stays healthy. - It took a while to figure out how to add a project that wasn't going to be packaged and shipped, but could also access shared code and what not. You'll see me defining the `AzureCoreSharedSources` property manually because that property is only set for official client libraries which have certain criteria like starting with "Azure.". * Add support for MSI credential in configuration (#18459) * selection marks allows null (#18452) * [Synapse] - General Available Release (#18252) * Generate code for 2020-12-01 * [Synapse] - General available release * Update AzSdk.RP.props Co-authored-by: Dongwei Wang <[email protected]> * Changes to refer new API version of Resource Mover Service (#18432) * Resource Mover generated code for new version. * Added tests. * Re ran generated.ps1 * Updates. * Updated AzSDK.props Co-authored-by: Ajay Gupta <[email protected]> * Update with refactoring. * Update with minor comments. * Update yml file * [MetricsAdvisor] Made AnomalyAlertConfiguration constructor parameterless (#18458) * [Storage] Webjobs update metapackage Changelog. (#18456) * [Synapse] Update swagger to aa19725fe79aea2a9dc580f3c66f77f89cc34563, bump to newest AccessControl API, add new Artifacts.Library API (#18343) - There is work here porting the existing AccessControl tests to the new API but I've also [Ignore]'ed the test. - The new API won't allow ID reuse for a month, and that's playing havoc w\ the recording system so far. * [synapse] Remove Library API for now from public API (#18481) * [Service Bus Client] Doc Comment Reformatting Test (#18485) The focus of these changes is to reformat the doc comments in a file that has been rendering incorrectly for the MS Docs site to test the theory that the indentation of the XML is influencing MS Docs formatting. * [Communication] - Ensure RestClients are created based on ClientOptions.ApiVersion (#18464) * Prepare for SB release (#18486) * Prepare for SB release * Update version * Remove <inheritdoc> from customized models to fix project warnings (#18484) * [MetricsAdvisor] Made NotificationHook constructors parameterless (#18475) * Updated jsom files. * Print assembly informational version (#18490) * [Core.Experimental] Improvements for Protocol Clients (#18412) [Core.Experimental] Improvements for Protocol Clients - Add a ProtocolClientOptions type for use by clients which have no additional options. This means we don't have to generate a type per client - Add a zero argument constructor to `DynamicJson` that behaves like DynamicJson.Object(), since we expect that `DynamicJson` will appear in API signatures and users will want to be able to `new` up the type. * Update AutoRest C# version to 3.0.0-beta.20210205.2 (#18460) * Azure Monitor Exporter: rename (#18494) * first pass on rename. compile successful * cleanup * Get ready for Azure.Core 1.9.0 (#18453) * Take reference on new TokenRequestContext (#18440) * Added a client consumption example (#18463) - In the board review, they requested that we add an example of how someone would consume the `AccessToken` in a client. No clients exist today, so we're keeping things hypothetical for now to make the point. * Define custom platform for HSM tests (#17925) Resolves #16483 * Increment package version after release of Azure.Core (#18502) * Downgrading Microsoft.Identity.Client dependency to 4.23 (#18492) We have been asked by a partner team to downgrade to Microsoft.Identity.Client 4.23 for our next beta release to align with their dependencies to avoid a resetting test coverage. We will again upgrade to the latest Microsoft.Identity.Client before the GA release of 1.4.0. * Ensure only a single action for a document is added to a batch (#18469) * Ensure only a single action for a document is added to a batch * Add test to validate batch splitting behavior * Add a test to verify case-sensitive nature of keys * Add Data Plane SDK for Azure Remote Rendering (#18159) This adds the .net SDK for Azure Remote Rendering. Tests are included which convert assets, start a session and confirm a number of failure cases behave correctly. * Rename a few properties in SearchIndexingBufferedSenderOptions (#18285) * Rename a few properties in SearchIndexingBufferedSenderOptions * Update samples and API using scripts * Update Batching.Publisher.BatchActionCount when handling a 413 * AppAuth 1.6.1 release (#18478) * Add the Remote Rendering package to the CI pipeline (#18523) When adding ARR, I missed the pipeline step which generates the package. * Remove flaky tests and increase CLI timeout (#18500) * Remove flaky tests and increase CLI timeout * Update nuget version in README.md (#18473) * Update nuget version in README.md * Addressing review comments + clean up for preview 3 release + update release date. Co-authored-by: Camilo Ramirez <[email protected]> * Update three pass services to enable live testing in sovereign clouds for multiple services (#18162) * change core,extension and search * remove core change * resolve ci error * expose length again (#18505) * Connection options fix (#18508) * Pass options to consumer client * fix * Update Azure.Core to 1.9.0 (#18538) * Prepare for Azure.Core.Experimental beta.8 release (#18537) * Prepare for release (#18514) * Prepare for release * Update changelog * Disable OS verification (#18518) * Disable OS verification * Update devops helper script based on changes in azure-sdk repo (#18540) - Sync devops helper script changes - Fix issue in SemVer script when using strict mode with Language not set Co-authored-by: Wes Haggard <[email protected]> * Log Key Vault Certificates content for live tests (#18558) Needed to diagnose a test failure where the operation is completed, but the status is still "inProgress". * [Text Analytics] Analyze design update (#18406) * Increment version for core releases (#18561) Increment package version after release of Azure.Core.Experimental * Update Azure.Core.Experimental to beta.8 (#18559) * Increment version for extensions releases (#18572) * [MetricsAdvisor] Made MetricFeedback class abstract (#18511) * Add sovereign cloud opt-in to tests weekly (#18488) * [Perf] [Stress] Add netcoreapp3.1, enable Server GC on net461 (#18577) * Also enable Server GC by default in stress projects * [Storage] Point to latest core. (#18581) * [MetricsAdvisor] Removed setters from collections (#18493) * Updating Backup SDK to 2021-01-01 (#18310) * Added generated changes for 2021-01-01 Backup SDK * Adding release info * Updated Test Cases * Fixing reference generation discrepency error * Using autorest 2.3.82 to generate code * Regenerating SDK based on latest Swagger * Remove search client helper method (#18578) * Remove helper method in SearchClient Callers can instantiate an instance of SearchIndexingBufferedSender in lieu of calling CreateIndexingBufferedSender(). * Update API file * Address PR comment for Inbound Endpoint E2E and added Nrp simulator change. * Fix typo. * [CDN] AFDX Update from latest swagger (#18587) * Update from latest swagger * Update version to 6.1.0 * [Storage] MessageDecodingFailed changelog. (#18585) - forgotten changelog entry. * [Event Hubs] Update Package Dependencies (#18560) The focus of these changes is to update package dependencies for the February release. * [Event Hubs Client] February Release Documentation (#18263) The focus of these changes is updating the samples, README, and change log to reflect the February milestone release. * [synapse] Reenable AccessControlClientLiveTests as LiveOnly. Mark others as RecordedTest (#18499) - Give up on making AccessControlClientLiveTests record-able and mark as LiveOnly - [RecordedTest] seems to be a flatly better [Test] for all RecordedTestBase derived tests, so use it * Increment version for eventhub releases (#18594) Increment package version after release of Azure.Messaging.EventHubs * [Event Hubs Client] Processor Release Prep (#18595) The focus of these changes is to prepare the Event Processor package for the Februrary release. * [Synapse][Artifacts] - Make some models public (#18515) * Make some models public * Fix Export API issue Co-authored-by: Dongwei Wang <[email protected]> * Update Contributing.md for Prepare Release Section (#18441) * Update Contributing.md * Remove eng/scripts/Prepare-Release.ps1 * Increment package version after release of Azure.Messaging.EventHubs.Processor (#18599) * [synapse] Correct version numbers (#18597) - AccessControl should be preview.3 (see https://www.nuget.org/packages/Azure.Analytics.Synapse.AccessControl/) - Artifacts should be preview.6 (since beta.1 should have been .5) (https://www.nuget.org/packages/Azure.Analytics.Synapse.Artifacts/) - Spark should be preview.4 (since beta1 should have been .3) (https://www.nuget.org/packages/Azure.Analytics.Synapse.Spark/) * add release date (#18602) * [synapse] Add release notes for Feb-2021 release (#18604) - Fold the misnamed beta.x releases into the next preview to prevent tool confusion * [TA] Update changelog (#18580) Fixes: #18563 * [synapse] Update release notes with concrete date (#18608) * Add an internal ObjectNotDisposedException (#18605) that we can throw/immediately catch from finalizers when an object wasn't properly disposed and there's pending work being lost or resources being leaked * remove Core proj reference (#18547) * [communication] prep release for common, identity, chat (#18586) * [Messaging Clients] README Updates (#18607) The focus of these changes is to update the README content for the messaging libraries to correct issues flagged during the MS Docs publishing validation. In addition, the Service Bus text detailing C# 8.0 and language versions has been expanded with some additional context. * Resolve archboard feedback (#18582) * Rename Encrypt/DecryptOptions to Encrypt/DecryptParameters * Make AdditionalAuthenticatedData read-only * Change parsing of Key Vault identifiers and implement IEquatable<T> * Remove proj ref to Core (#18619) * [Perf] Prevent divide-by-zero for incomplete threads (#18512) * Increment package version after release of Azure.Messaging.EventGrid (#18620) * Improve the dev feed section (#18462) * Improve the dev feed section * Remove csproj option. Remove multiple feed refs. * Remove csproj file ref * Code formatting * Change aka to full url * Increment version for formrecognizer releases (#18622) Increment package version after release of Azure.AI.FormRecognizer * Update to close pending comments. * [MetricsAdvisor] Updated Create methods to return whole models instead of ID (#18491) * Adding "Thread safety" and "Additional concepts" sections to our client's READMEs (#18354) * [ContainerInstance] added ci.yml file for the pipeline. (#18349) * Update Microsoft.Extensions.Azure to 1.1.0-beta.2 (#18574) * update * refs * Make proj ref and mark as extension library * service bus Co-authored-by: JoshLove-msft <[email protected]> * Fixing error response for CosmosDB 2020-06-01-preview (#18399) * Fixing error response for CosmosDB 2020-06-01-preview * Updating SDK using merged swagger * Updating generate.ps1 * Fixing assembly version * Removing config file tag * Rerunning with correct version of autorest.csharp * Version 2.3.82 * Rerunning generate.ps1 * Update changelog for release (#18625) * Fixed Github IO to use .NET display name (#18623) * Increment version for identity releases (#18628) Increment package version after release of Azure.Identity * Increment version for communication releases (#18627) * Increment package version after release of Azure.Communication.Identity * Increment package version after release of Azure.Communication.Common * Increment package version after release of Azure.Communication.Chat * Increment version for metricsadvisor releases (#18629) Increment package version after release of Azure.AI.MetricsAdvisor * Updated storage changelogs (#18617) * Prepare Azure.Search.Documents 11.2.0 GA release (#18621) * Prepare Azure.Search.Documents 11.2.0 GA release * Add public changes in the CHANGELOG * Add a line to show inclusion of all beta changes in GA * Increment version for servicebus releases (#18624) Increment package version after release of Azure.Messaging.ServiceBus * Add release date (#18636) * Prepare beta release of Key Vault SDKs (#18635) * Increment version for storage releases (#18637) * Azure Purview dot net SDK (#18237) * Azure Purview dot net SDK * Add Purview to build path * Update code location to make generation file * Add readme * move readme * Update SDK generation file location * Update to add src and test folder * Fix readmes * Update nuget version, address minor pr comments * Changes ExampleId type to long (#18543) * Changes ExampleId type to long * Fixes build errors * Validates skip and take not equal null * retrigger checks Co-authored-by: Younes <[email protected]> * Increment version for eventhub releases (#18638) Increment package version after release of Microsoft.Azure.WebJobs.Extensions.EventHubs * [Azure.Core.Experimental] Package Ref to Core (#18618) The focus of these changes is to shift to a pacakge reference to Azure.Core, allowing the dependency to bind to the latest GA release. * Prepare MR Authentication for pre-release (#18613) This change removes the nullable awareness since that is the current guidance. I've also changed the prerelase tag to match current guidance. * Update CHANGELOG.md (#18658) * Changes from running the release script (#18659) These are changes related to preparing to release the package. * Update AuthenticationRecord and TokenCache Samples (#18639) 1. The `using` declaration is scoped to the method, so the stream was not closed before it needed to be read. Changed to `using(){}` 2. Added line wrapping so devs don't have to horizontal scroll when viewing the samples. I ran `dotnet test Azure.Identity.sln` before pushing and it passed. * Changes from running the release script. (#18661) * Increment version for synapse releases (#18653) * Increment package version after release of Azure.Analytics.Synapse.Spark * Increment package version after release of Azure.Analytics.Synapse.Monitoring * Increment package version after release of Azure.Analytics.Synapse.ManagedPrivateEndpoints * Increment package version after release of Azure.Analytics.Synapse.Artifacts * Increment package version after release of Azure.Analytics.Synapse.AccessControl * Updated to namespace. * Update Azure.Core.Experimental to beta.9 (#18664) * Add SecretReferenceConfigurationSetting, FeatureFlagConfigurationSetting and AddSyncToken (#18487) * Increment version for textanalytics releases (#18665) Increment package version after release of Azure.AI.TextAnalytics * Fix description in csproj. (#18671) * Fix #16362 Add visual to the documentation to explain the mined opinions structure and naming better (#18525) Co-authored-by: NourEldin Yasser <[email protected]> * [textanalytics] fix AnalyzeOperationBatchWithStatisticsTest (#18516) Co-authored-by: NourEldin Yasser <[email protected]> * Sync eng/common directory with azure-sdk-tools for PR 1393 (#18666) * Update instructions to run the sample search query (#18654) * Update instructions to run the sample search query * Update AutoRest C# version (#18668) * Update assertion for ResourceGuid. * Fix #17811 Improve samples for Analyze (#18524) * [textanalytics] rename Sample_AnalyzeOperation.md to Sample_AnalyzeBatchActions.md * [textanalytics] Fix AnalyzeOperation Sample Source Code * [textanalytics] update code snippets in md files from samples source code * [textanalytics] remove polling samples and fix snippets references in readme.md and AnalyzeBatchActions.md Co-authored-by: NourEldin Yasser <[email protected]> * Update version of Azure.Search.Documents to 11.3.0-beta.1 (#18672) * mixed reality csharp sdk for 2021-01-01 API (#18448) * mixed reality charp sdk * update csproj * Add missing files * Update READMEs for Key Vault SDK betas (#18677) * Add session idle timeout option for session processor (#18640) * Add session idle timeout option for session processor * Fix docs * PR fb * Sync eng/common directory with azure-sdk-tools for PR 1303 (#18684) * Add PrepareRelease Script * Update Package-Properties.ps1 * Update Collect-ChangeLog Script * Update Collect-ChangeLog.ps1 and Collect-Unreleased.ps1 * Update GeneralReleaseNotesParser.ps1 script * Change GeneralReleaseNotesParser logic * Change GeneralReleaseNotesParser.ps1 logic * Remoce GeneralReleaseNotesParsercopy * Update collect changelog and generalreleasenotes parser logic * Refine CollectChangelog Logic * Add logic for filtering collected changelog * Add Filter-ReleaseHighlights function * Update General ReleasNotes Logic * Delete GeneralReleaseNotesLogic.ps1 * Update Collect ChangeLogLogic Co-authored-by: Chidozie Ononiwu <[email protected]> * Increment version for core releases (#18662) * Increment version for mixedreality releases (#18689) * Increment package version after release of Azure.MixedReality.Authentication * Increment package version after release of Azure.MixedReality.RemoteRendering * [synapse] Fix StartCreateSparkStatement API visibility (#18663) * Add import and non-sparse parameter functionality to job matrix generation (#18660) * [synapse] Fix AccessControl LIVE sample test (#18606) * [synapse] Changelog for Spark StartCreateSparkStatement release patch (#18693) * Sync eng/common directory with azure-sdk-tools for PR 1358 (#18686) * Extend Get-GitHubPullRequest to return only files * Add check to prevent eng/common changes that are not proper syncs * Update eng/common/pipelines/templates/steps/eng-common-workflow-enforcer.yml Co-authored-by: Wes Haggard <[email protected]> * Update eng/common/pipelines/templates/steps/eng-common-workflow-enforcer.yml Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Chidozie Ononiwu <[email protected]> Co-authored-by: Chidozie Ononiwu <[email protected]> Co-authored-by: Wes Haggard <[email protected]> * Update AutoRest C# version (#18674) * Enable passing devops pat for work item scripts (#18698) Co-authored-by: Wes Haggard <[email protected]> * Increment package version after release of Azure.Analytics.Synapse.Spark (#18709) * Change script parameters to boolean (#18708) Co-authored-by: Chidozie Ononiwu <[email protected]> * Add net5.0 to smoke tests. Fix dependency issues. (#18696) * Add net5.0 to smoke test target frameworks * Update azure.messaging.eventhubs to 5.3.0 in smoke tests * Remove daily version override for eventhubs in smoke test * Add check to catch delinquent eng/common changes (#18311) * Rename Microsoft.OpenTelemetry to Azure.Monitor.OpenTelemetry (#18702) * Revert "Add check to catch delinquent eng/common changes (#18311)" (#18710) This reverts commit 8ccc759. * Add variable setting warning level to warn (#18711) * [Communication] Remove administration package (#18655) * [Communication] Remove administration package * Remove the admin package from the ci.yml file * Update incorrect links in ReadMe files Co-authored-by: Jesse Squire <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Azure SDK Bot <[email protected]> Co-authored-by: Heath Stewart <[email protected]> Co-authored-by: JoshLove-msft <[email protected]> Co-authored-by: matusthemostlygreat <[email protected]> Co-authored-by: Victor XField <[email protected]> Co-authored-by: tg-msft <[email protected]> Co-authored-by: Christopher Scott <[email protected]> Co-authored-by: Mariana Rios Flores <[email protected]> Co-authored-by: Rahul Singh <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: praveenkuttappan <[email protected]> Co-authored-by: Larry Li <[email protected]> Co-authored-by: Kamil Sobol <[email protected]> Co-authored-by: Scott Schaab <[email protected]> Co-authored-by: Craig Treasure <[email protected]> Co-authored-by: Mitch Denny <[email protected]> Co-authored-by: Shivangi Reja <[email protected]> Co-authored-by: Yugang Wang <[email protected]> Co-authored-by: Alex Ghiondea <[email protected]> Co-authored-by: Heath Stewart <[email protected]> Co-authored-by: Amanda Nguyen <[email protected]> Co-authored-by: praveenkuttappan <[email protected]> Co-authored-by: XField <[email protected]> Co-authored-by: Paul Batum <[email protected]> Co-authored-by: kinelski <[email protected]> Co-authored-by: Sima Zhu <[email protected]> Co-authored-by: Chidozie Ononiwu <[email protected]> Co-authored-by: Reza Jooyandeh <[email protected]> Co-authored-by: Sima Zhu <[email protected]> Co-authored-by: Suhas Mehta <[email protected]> Co-authored-by: Matthew Burleigh <[email protected]> Co-authored-by: Azad Abbasi <[email protected]> Co-authored-by: Robert Gao <[email protected]> Co-authored-by: Allen Zhang <[email protected]> Co-authored-by: Adam Sandor <[email protected]> Co-authored-by: Sean McCullough <[email protected]> Co-authored-by: nmaredia <[email protected]> Co-authored-by: Camilo Ramirez <[email protected]> Co-authored-by: Camilo Ramirez <[email protected]> Co-authored-by: Jingshu923 <[email protected]> Co-authored-by: Chris Hamons <[email protected]> Co-authored-by: Kushagra Thapar <[email protected]> Co-authored-by: Pavel Krymets <[email protected]> Co-authored-by: PoAn (Baron) Chen <[email protected]> Co-authored-by: Chenglong Liu <[email protected]> Co-authored-by: Chenglong Liu <[email protected]> Co-authored-by: David R. Williamson <[email protected]> Co-authored-by: Mike Harder <[email protected]> Co-authored-by: Dongwei Wang <[email protected]> Co-authored-by: Dongwei Wang <[email protected]> Co-authored-by: ajaygit158 <[email protected]> Co-authored-by: Ajay Gupta <[email protected]> Co-authored-by: Matt Ellis <[email protected]> Co-authored-by: Timothy Mothra <[email protected]> Co-authored-by: Mohit Chakraborty <[email protected]> Co-authored-by: Malcolm Tyrrell <[email protected]> Co-authored-by: Anais Miller <[email protected]> Co-authored-by: Yun Lu (MSFT) <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]> Co-authored-by: Anjor S. Hemant <[email protected]> Co-authored-by: Chidozie Ononiwu <[email protected]> Co-authored-by: Dominik <[email protected]> Co-authored-by: Jon Gallant <[email protected]> Co-authored-by: LucasYao93 <[email protected]> Co-authored-by: nichatur <[email protected]> Co-authored-by: annoakes <[email protected]> Co-authored-by: MahmoudYounes <[email protected]> Co-authored-by: Younes <[email protected]> Co-authored-by: NourEldin Yasser <[email protected]> Co-authored-by: NourEldin Yasser <[email protected]> Co-authored-by: roytan-microsoft <[email protected]> Co-authored-by: Rajkumar Rangaraj <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Fixes #15476